----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110833/ -----------------------------------------------------------
(Updated June 12, 2013, 2:08 p.m.) Review request for KDE Frameworks and David Faure. Changes ------- Don't include kglobalsettings from kiconloader Description ------- As we discussed in the IRC meeting, I removed the KIconLoader dependency to KWidgets (so that we can make KWidgets depend on KIconLoader). I'm unsure if it's the best approach, so I tried to do the simplest implementation I could. Don't hesitate to suggest changes. Diffs (updated) ----- staging/kiconthemes/src/CMakeLists.txt 82828dc staging/kiconthemes/src/kiconloader.h e4423eb staging/kiconthemes/src/kiconloader.cpp 795ec93 staging/kwidgets/src/CMakeLists.txt 2775463 staging/kwidgets/src/utils/kglobalsettings.h 5fb1d9b staging/kwidgets/src/utils/kglobalsettings.cpp 53b648e Diff: http://git.reviewboard.kde.org/r/110833/diff/ Testing ------- it builds Thanks, Aleix Pol Gonzalez
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel