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
hspell: fix cross-platform build issues #96896
Conversation
I notice that this doesn't have a maintainer listed. Would you be interested in taking it over? |
Sure, added myself as a maintainer, but in all fairness, I'm not passionate about this at all. |
@GrahamcOfBorg build hspell |
@GrahamcOfBorg eval |
hspell uses a couple of binaries it generates during it's own build. These will cause the build to fail later in cross-platform builds as these binaries have the wrong executable format. Using a hspell to bootstrap it's own build natively resolves this issue.
@GrahamcOfBorg build hspell |
Is there any way to contribute patches upstream or is this project pure cathedral style? |
I am neither familiar with the term |
Cathedral style means software that is open source, but largely or entirely directed by one person or group. It’s less about whether they accept outside patches, but more about whether outsiders can be part of changing how the program works or coming up with its overall direction. The term comes from a famous essay about open source software development called The Cathedral and the Bazaar. |
Got it, thanks for clarifying. |
I tried updating the package to the latest version: https://github.com/jtojnar/nixpkgs/pull/new/hspell-1.4 Could you please try if the situation is not better there? |
I tried that already, but it not only has the same issues, it actually introduces new ones when linking against |
@GrahamcOfBorg build hspell |
Motivation for this change
hspell uses a couple of binaries it generates during it's own build.
These will cause the build to fail later in cross-platform builds as
these binaries have the wrong executable format.
Using a hspell to bootstrap it's own build natively resolves this issue.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)