----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127033/#review92654 -----------------------------------------------------------
Fix it, then Ship it! src/plugins/hunspell/hunspelldict.cpp (line 48) <https://git.reviewboard.kde.org/r/127033/#comment63156> should be QFile::encodeName(QCoreApplication::applicationDirPath()), for symmetry. src/plugins/hunspell/hunspelldict.cpp (line 63) <https://git.reviewboard.kde.org/r/127033/#comment63157> Simpler: dirPath.chop(1) But wouldn't it be simpler to make "dic" not have a trailing slash, and add +'/' in the composeDictName function? - David Faure On Feb. 22, 2016, 8:59 p.m., Kåre Särs wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/127033/ > ----------------------------------------------------------- > > (Updated Feb. 22, 2016, 8:59 p.m.) > > > Review request for KDE Frameworks, Laurent Montel and Martin Tobias Holmedahl > Sandsmark. > > > Repository: sonnet > > > Description > ------- > > The default search path for hunspell dictionaries is /usr/share/hunspell/, > but that is not a very common directory on windows ;) > > For Windows and Mac there are some extra search paths defined in side > #ifdefs, but they are hardcoded to the LibreOffice dictionary directories. > Unfortunately the extra Windows path points to a 32bit LibreOffice 5 on a 64 > bit Windows directory. This patch adds <app dir>/../share/hunspell/ to the > search path in case the default is not found. > > If wanted I can add a CMake define for specifying the path relative to the > application directory. > > > Diffs > ----- > > src/core/guesslanguage.cpp e8deb90 > src/plugins/hunspell/hunspellclient.cpp 3da7219 > src/plugins/hunspell/hunspelldict.cpp bc5c314 > > Diff: https://git.reviewboard.kde.org/r/127033/diff/ > > > Testing > ------- > > Compiled Sonnet and Kate on Windows and got spelling to work :) > > It is now possible to add dictionaries to a relocatable <App > dir>/../share/hunspell/ and get spelling to work without depending on > LibreOffice installation on Windows. > > > Thanks, > > Kåre Särs > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
