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
hunspellDict.fr-any: Link fr-moderne.dic
to fr_FR.dic
#47153
Conversation
fr-toutesvariantes.dic
to fr_FR.dic
to hunspellDict.fr-any
fr-toutesvariantes.dic
to fr_FR.dic
Success on x86_64-linux (full log) Attempted: hunspellDicts.fr-any, ibus-engines.typing-booster Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: hunspellDicts.fr-any, ibus-engines.typing-booster Partial log (click to expand)
|
d482728
to
c2fa5a7
Compare
c2fa5a7
to
c7988e1
Compare
fixed and rebased onto master. |
fr-toutesvariantes.dic
to fr_FR.dic
fr-moderne.dic
to fr_FR.dic
c7988e1
to
2fca82e
Compare
Ping @symphorien @jtojnar |
@Ma27 I don't see any reasons here for this not to be merged. 19.03 has past now, can we have the release note in 19.09? |
Some packages like `ibus-engines.typing-booster` require the dictionary `fr_FR.dic` to provide proper support for the french language. Until now the hunspell package set of nixpkgs didn't provide this dictionary. It has been recommended to use `fr-moderne` as base and link `fr_FR.dic` from it as done by other distros such as ArchLinux. See NixOS#46940 (comment) Fixes NixOS#46940
This has been postponed[1] because of an unclear state of the french dictionary provided by hunspell[2]. [1] NixOS#46779 (comment) [2] NixOS#46940 (comment)
2fca82e
to
77d8f5c
Compare
@worldofpeace fixed |
Thanks for fixing this @Ma27 ✨ Especially so because you noted that you don't speak french. |
Motivation for this change
See #46940
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)