Re: Review Request 122987: Allow user to specify path to myspell dictionary files

2015-03-17 Thread Rolf Eike Beer
> On März 17, 2015, 1:07 nachm., Laurent Montel wrote: > > src/plugins/hunspell/hunspellclient.cpp, line 27 > > > > > > #include "..." > > we use local file. No, the file is in an include path, not in the sam

Re: Review Request 122987: Allow user to specify path to myspell dictionary files

2015-03-17 Thread Eugene Shalygin
> On Березень 17, 2015, 6:05 після полудня, Christophe Giboudeaux wrote: > > src/plugins/hunspell/CMakeLists.txt, line 7 > > > > > > So you change an hardcoded path by a user chosen hardcoded one ? > > > >

Re: Review Request 122987: Allow user to specify path to myspell dictionary files

2015-03-17 Thread Christophe Giboudeaux
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122987/#review77647 --- src/plugins/hunspell/CMakeLists.txt

Re: Review Request 122652: Use correct default value when UDS_ACCESS/UDS_FILE_TYPE is not set

2015-03-17 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122652/#review77642 --- Ship it! Ship It! - David Faure On March 17, 2015, 4:17 p.

Re: Review Request 122652: Use correct default value when UDS_ACCESS/UDS_FILE_TYPE is not set

2015-03-17 Thread David Faure
> On March 17, 2015, 4:38 p.m., David Faure wrote: > > Ship It! Yes, both in kdelibs 4 and in the kio framework. - David --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122652/#review776

Re: Review Request 122987: Allow user to specify path to myspell dictionary files

2015-03-17 Thread Laurent Montel
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122987/#review77636 --- Ok for me. Just wait that maintainer gives you a "Ship it" -

Re: Review Request 122987: Allow user to specify path to myspell dictionary files

2015-03-17 Thread Eugene Shalygin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122987/ --- (Updated Березень 17, 2015, 2:09 після полудня) Review request for KDE Fr

Re: Review Request 122987: Allow user to specify path to myspell dictionary files

2015-03-17 Thread Laurent Montel
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122987/#review77631 --- I am not maintainer of this module but I can put a "+1" - Lau

Re: Review Request 122987: Allow user to specify path to myspell dictionary files

2015-03-17 Thread Laurent Montel
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122987/#review77630 --- src/plugins/hunspell/hunspellclient.cpp

Re: Review Request 122987: Allow user to specify path to myspell dictionary files

2015-03-17 Thread Eugene Shalygin
> On Березень 17, 2015, 1:41 після полудня, Luigi Toscano wrote: > > (not entitled to give a ship it, but) have you checked if this is still > > relevant for the Sonnet framework? > > Luigi Toscano wrote: > Ups, this review is for sonnet; I was confused by the group (please use > kdeframew

Re: Review Request 122987: Allow user to specify path to myspell dictionary files

2015-03-17 Thread Eugene Shalygin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122987/ --- (Updated Березень 17, 2015, 2:05 після полудня) Review request for KDE Fr

Re: Review Request 122987: Allow user to specify path to myspell dictionary files

2015-03-17 Thread Eugene Shalygin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122987/ --- (Updated Березень 17, 2015, 1:50 після полудня) Review request for KDE Fr

Re: Review Request 122987: Allow user to specify path to myspell dictionary files

2015-03-17 Thread Luigi Toscano
> On March 17, 2015, 1:41 p.m., Luigi Toscano wrote: > > (not entitled to give a ship it, but) have you checked if this is still > > relevant for the Sonnet framework? Ups, this review is for sonnet; I was confused by the group (please use kdeframeworks instead of kdelibs for Frameworks-relate

Re: Review Request 122987: Allow user to specify path to myspell dictionary files

2015-03-17 Thread Luigi Toscano
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122987/#review77622 --- (not entitled to give a ship it, but) have you checked if this

Re: Review Request 122987: Allow user to specify path to myspell dictionary files

2015-03-17 Thread Laurent Montel
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122987/#review77619 --- src/plugins/hunspell/hunspellclient.cpp

Re: Review Request 122987: Allow user to specify path to myspell dictionary files

2015-03-17 Thread Eugene Shalygin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122987/ --- (Updated Березень 17, 2015, 1:22 після полудня) Review request for kdelib

Re: Review Request 122987: Allow user to specify path to myspell dictionary files

2015-03-17 Thread Laurent Montel
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122987/#review77617 --- src/plugins/hunspell/CMakeLists.txt

Review Request 122987: Allow user to specify path to myspell dictionary files

2015-03-17 Thread Eugene Shalygin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122987/ --- Review request for kdelibs. Repository: sonnet Description --- Not

Changing the licence of parley's editor model classes from GPL to LGPL

2015-03-17 Thread Rahul Chowdhury
Hi, I, and Inge Wallin ( CC'ed in this email ), have been working on a project on moving the editor of parley into libkeduvocdocument. For that we have moved the model classes of parley's editor into keduvoc, and used them in the entire codebase of parley instead of the old model classes. But on