Greetings, Matt Seitz! >>> https://bugzilla.redhat.com/show_bug.cgi?id=494084 >>> >>> Does it make sense for Cygwin to follow Fedora's lead, and install >>> "aspell-en" by default when "aspell" is installed? >> >> No, it makes sense to report this issue to Git maintaners.
> OK, I can do that. > However, based on the Fedora bug reports, it sounds like this could be an > issue for other "aspell" client applications. It appears that the normal > behavior for "aspell" clients is to report an error if the "default" > dictionary is specified and no dictionary is present. Application should not error out on mistakes in a third-party module configuration, that is not even required for application to run, and merely providing a cosmetic functionality. > So it seems like the options are: > 1. Change git-gui and other "aspell" based applications to suppress their > error message when the "default" dictionary is specified and no dictionary > is present. Specified where? If it's a configuration in Git-GUI, it must report it as inconsistency and suggest the way to amend it. If it's a configuration in aspell, and Git-GUI merely asking for default behavior, then again, Git-GUI should check, if the module can work at all, before trying to use it and throw errors at the user. > 2. Change "setup.exe" to install the "default" dictionary (i.e., the > dictionary for the current locale) automatically when "aspell" is installed. Unlikely to happen. Even if do, it would only be a suggestion. Unless you take the burden of implementing full APT functionality into cygwin setup.exe. > 3. Change "setup.exe" to remind the user to install the "default" > dictionary when "aspell" is installed. > 4. Keep the current behavior, and let the end-user figure out what went > wrong and how to fix it. > Option 2 seems to be the solution that Fedora went with. -- WBR, Andrey Repin (anrdae...@freemail.ru) 08.11.2012, <22:59> Sorry for my terrible english... -- Problem reports: http://cygwin.com/problems.html FAQ: http://cygwin.com/faq/ Documentation: http://cygwin.com/docs.html Unsubscribe info: http://cygwin.com/ml/#unsubscribe-simple
