-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
This issue was moved to a discussion.
You can continue the conversation there. Go to discussion →
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Misbeavings with target x86_64-unknown-linux-musl #1603
Comments
You have to tell helix where to find the runtime directory, so you'll need something like ENV HELIX_RUNTIME="/helix-${VERSION}/runtime" |
Ah cool, you got it to build on musl? Linking back to an older thread where there was some issues #1028 |
Hello @dead10ck, and thanks for your advice. I made some minor edits to the Dockerfile and I added the following line: ENV PATH=/helix-${VERSION}:"$PATH" ... so that the helix editor can be invoked simply issuing export HELIX_RUNTIME="/helix-${VERSION}/runtime" |
Ah I see, maybe it's related to #1028 then. |
Hello @archseer, yes, I setup a Dockerfile in a WIP github repo. docker pull lanceschi/helix-editor-musl:latest ... and have a look: docker run --rm -it lanceschi/helix-editor-musl:latest /bin/ash You can also have a look at the logs of the Github Dockerfile build action. |
I’ve packaged Helix in Alpine Linux (package helix), so you don’t have to build it yourself anymore. It’s currently in the testing repo, I’m gonna move it to the community repo soon. |
Hello @jirutka, thanks a lot! I had a try issuing the following from a echo "@testing http://dl-cdn.alpinelinux.org/alpine/edge/testing" >> /etc/apk/repositories
apk add helix@testing The post message install note reports: fetch http://dl-cdn.alpinelinux.org/alpine/edge/testing/x86_64/APKINDEX.tar.gz
(1/1) Installing helix@testing (0.6.0-r0)
Executing helix-0.6.0-r0.post-install
*
* Tree-sitter grammars for syntax highlighting are packaged separately.
* Install either selected tree-sitter-<lang> packages for the languages you
* use or a "virtual package" tree-sitter-grammars to install all of them.
* How am I supposed to install those? Another thing I noticed checking the helix version number: $ hx -V
helix v3.15.0-5432-gd33618c49f Isn't it supposed to be |
I don’t understand what’s unclear on this. They are packaged separately, i.e. there are packages tree-sitter-rust, tree-sitter-ruby etc. which you can install to get syntax highlighting for the given languages. Or you can install
Thanks, I’ll fix this. |
@jirutka Thank you for packaging helix! 🎉
Yeah this was my concern when the change was introduced in #957. I'll revert that and only pull the commit sha from the repository. That won't fully fix it but at least the version will be correct. EDIT: d3221b0 |
Thanks 💯 for the clarification @jirutka. I followed your advices: echo "@edge https://dl-cdn.alpinelinux.org/alpine/edge/community/" >> /etc/apk/repositories
apk add tree-sitter-javascript@edge Now I gained syntax highlighting for Any clues? |
What terminal are you using? Does it support true color? |
You’re welcome!
You can test if there’s a |
I can confirm @archseer’s findings, I also see black background. I didn’t know it was supposed to be purple by default, so I didn’t initially consider it a bug. However, I dunno what can be the cause. |
I just tried it on my Alpine desktop in Kitty terminal and I see a purple background. Relevant env. variables:
|
|
Weird, wezterm should be already setting the COLORTERM var. |
No, actually it makes sense @archseer. I launch WezTerm from the host computer where I faced the same issue with a CentOS v7.x powered machine as well. |
@jirutka is it possible to see the Dockerfile or CLI command sequence you're using to build helix? |
testing/helix in the Alpine aports repository. However, you will not find anything special here. Maybe the key difference is that you build it with the stock rustc instead of rustc from Alpine package. |
This issue was moved to a discussion.
You can continue the conversation there. Go to discussion →
Reproduction steps
I managed to build helix editor v0.6.0 with target set to
x86_64-unknown-linux-musl
.Here's the Dockerfile. If I extract the binary from the built image and I run it on my Ubuntu 20.04.3 LTS desktop for instance, it works as expected. It did not under the Alpine Linux v3.15.0 container. The theme doesn't seem to be applied as well as the syntax highlighting.
Build the above Dockerfile:
... and run the container with the following:
From within the running container download an example file:
Open it verbosely:
Here's a screenshot:
Here the content of Alpine Linux file
~/.cache/helix/helix.log
:Here's the screenshot of the same file opened with the same
musl
binary on the host Ubuntu 20.04.3 LTS:Here's the Ubuntu log
~/.cache/helix/helix.log
:Environment
Thanks 💯 for the awesome app!!!
The text was updated successfully, but these errors were encountered: