> On Oct. 17, 2015, 11:22 a.m., David Faure wrote: > > autotests/fallbackconfigresourcestest.cpp, line 32 > > <https://git.reviewboard.kde.org/r/125598/diff/2/?file=410731#file410731line32> > > > > unused?
Fixed ;=) Beside that, ok to go in? Should it be :kconfig5/ or :kconfig/ like asked by Matthew? - Christoph ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125598/#review86951 ----------------------------------------------------------- On Oct. 17, 2015, 12:01 p.m., Christoph Cullmann wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/125598/ > ----------------------------------------------------------- > > (Updated Oct. 17, 2015, 12:01 p.m.) > > > Review request for KDE Frameworks and David Faure. > > > Repository: kconfig > > > Description > ------- > > Like KXMLGui, KConfig has now a fallback to config files bundled with the > application in resources. > > Like kxmlgui, its in :/kconfig5. > > This allows e.g. KTextEditor deployment without any installed files beside > the library itself and should easy the shipping of default configs for > non-linux/unix platforms. > > Question is if :/kconfig5 is that nice a prefix, at least it is unique and > matches what we do with kxmlgui5. > > > Diffs > ----- > > autotests/CMakeLists.txt 8213bc4 > autotests/fallbackconfigresources.qrc PRE-CREATION > autotests/fallbackconfigresourcestest.cpp PRE-CREATION > src/core/kconfig.cpp 7f03869 > > Diff: https://git.reviewboard.kde.org/r/125598/diff/ > > > Testing > ------- > > Added an unit test to show that the fallback resource is found and read. > > > Thanks, > > Christoph Cullmann > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel