----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126994/#review92106 -----------------------------------------------------------
I suppose that getting rid of KDELibs4Support isn't a bad thing at all if it doesn't provide crucial/central functionality. PS: you're welcome :) - René J.V. Bertin On Feb. 5, 2016, 9:16 p.m., Aleix Pol Gonzalez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/126994/ > ----------------------------------------------------------- > > (Updated Feb. 5, 2016, 9:16 p.m.) > > > Review request for Plasma and René J.V. Bertin. > > > Repository: plasma-workspace > > > Description > ------- > > As I saw in another RR, the simple fact of using KDELibs4Support in part of a > project makes the whole project rely on Qt4Support. That's bad especially in > cases where KDELibs4Support is optional, because it will lead to > unpredictable results. > > > Diffs > ----- > > applets/systemtray/plugin/host.cpp e529aeb > components/shellprivate/widgetexplorer/kcategorizeditemsviewmodels.cpp > cba87ec > components/shellprivate/widgetexplorer/kcategorizeditemsviewmodels_p.h > e9d7ef3 > components/shellprivate/widgetexplorer/plasmaappletitemmodel.cpp 8a5ba91 > components/shellprivate/widgetexplorer/plasmaappletitemmodel_p.h 20fb98e > components/shellprivate/widgetexplorer/widgetexplorer.cpp 7816614 > dataengines/CMakeLists.txt 9ee93d8 > dataengines/dict/dictengine.cpp d851268 > dataengines/mpris2/mpris2engine.h ea49515 > dataengines/mpris2/multiplexedservice.h 2ab4e6b > dataengines/mpris2/playercontainer.cpp 6c4c875 > dataengines/share/shareprovider.cpp ac66fd3 > dataengines/statusnotifieritem/statusnotifieritemsource.cpp 097f110 > drkonqi/bugzillaintegration/bugzillalib.cpp 802c5fb > drkonqi/bugzillaintegration/reportassistantpages_bugzilla.cpp e60fb14 > drkonqi/bugzillaintegration/reportassistantpages_bugzilla_duplicates.cpp > 4f8f4ea > drkonqi/drkonqi.cpp b12c118 > libtaskmanager/abstractgroupableitem.cpp 9b86cf1 > libtaskmanager/groupmanager.cpp 12b18b0 > libtaskmanager/strategies/manualgroupingstrategy.cpp b37ac35 > libtaskmanager/strategies/programgroupingstrategy.cpp 6b91bfe > libtaskmanager/taskactions.cpp d1ddf9c > libtaskmanager/taskactions_p.h 535ea5f > libtaskmanager/taskitem.cpp 3b2a418 > libtaskmanager/tasksmodel.h 8decf0b > libtaskmanager/tasksmodel.cpp 0abdd93 > runners/bookmarks/faviconfromblob.cpp 540db9c > runners/bookmarks/tests/testchromebookmarks.cpp 718e765 > shell/currentcontainmentactionsmodel.h 0c27a20 > shell/currentcontainmentactionsmodel.cpp d442985 > shell/kidenticongenerator.cpp 4e4b25a > shell/panelview.cpp 0eaf553 > shell/scripting/backportglobal.h 988b14e > shell/scripting/containment.cpp 5e42bee > shell/scripting/widget.cpp 984e666 > wallpapers/image/backgroundlistmodel.h d639b03 > wallpapers/image/backgroundlistmodel.cpp b4f81da > wallpapers/image/image.h 295ba22 > > Diff: https://git.reviewboard.kde.org/r/126994/diff/ > > > Testing > ------- > > Builds, tests pass. > > > Thanks, > > Aleix Pol Gonzalez > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel