-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
feat: add helix #6338
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
base: master
Are you sure you want to change the base?
feat: add helix #6338
Conversation
|
Thanks for contributing. helix is candiate to be added to synocli-file package. But there is a cross compilation error when building the tree-sitter crate. I already found this issue when I tried to build helix 1 1/2 year ago, .... |
|
This is the build error for aarch64-7.1 The wrong CFLAG |
|
I added I tried changing things like |
|
@etcusrvar I found a solution to cross compile helix. I will continue your work here... |
- update helix to latest version - fix cross compilation - fix build of language grammar modules
Description
Add helix- https://helix-editor.com
Remarks
This needed some tweaking, because the upstream releases are not cross compiled but created on native systems (for x64 and aarch64).
By default helix is first built for the host arch and not for the target. And an error in cc crate does not detect the correct target arch (this is fixed by include and patching the cc crate source).
Another impediment is the build of the tree-sitter grammar modules. Those are not built by rust but by native c compiler (under the control of cargo build system). This is fixed by explicit cross tool definitions.
helix is not a small tool (due to the language processing modules) so it is not an option to add it to synocli-file but to create its own package.
Checklist
all-supportedcompleted successfullyType of change
draft PR status
This currently builds, installs, and works correctly on avoton-7.1.
all-supporteddoesn't yet build without errors.