Set aspell's data-dir through ASPELL_CONF in a wrapper #24425
Conversation
|
This only allows installing dicts with nix-env :-/ I have them in configuration.nix, so that will break for me. Is there no other way to fix this for non-NixOS users? Has anyone looked into making aspell take a list of dict-dirs? |
|
How about an option to set ASPELL_CONF only if it's not set? |
Sounds good to me. |
|
Done! |
|
If you want to change makeWrapper I think it should be in a separate commit. Perhaps even separate PR, for visibility. It's a mass rebuild so it'd have to go via the staging branch. I don't think the need for a --default option in makeWrapper has come up before, so the question is whether this is something we generally need. Do you think there will be other users than "aspell"? Since you're adding a wrapper... how about writing a custom wrapper for aspell which iterates over $NIX_PROFILES and sets $ASPELL_CONF accordingly (like NixOS does)? Benefits: works no matter how aspell was installed and we can get rid of the ASPELL_CONF logic in NixOS. Hmm, I see $NIX_PROFILES isn't set in my Ubuntu+Nix. Is it set on macOS? One could hardcode the profile paths in the wrapper until the profile scripts on all platforms set $NIX_PROFILES. |
|
Sounds better! I will try that. |
|
I'm not very confident that this Is this what you mean. If it is not, could you direct me to an example? |
|
Yes, that's what I had in mind :-) Did you test it? Please prefix commit message with "aspell:". |
if the user didn't set ASPELL_CONF, use it to point to the dictionary location by first looking at NIX_PROFILE directories and then using $HOME/.nix-profile. See #1000
|
Cool! |
|
Looks good to me. |
This makes aspell find the dictionaries in the
user directory, and is essentially what the
documentation suggested.
See #1000
Motivation for this change
#1000
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandboxinnix.confon non-NixOS)
nix-shell -p nox --run "nox-review wip"./result/bin/)