----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113920/#review44043 -----------------------------------------------------------
Ship it! Tested it here locally. Building plasma-shell against it and running it is a pretty good test to pick up regressions, on top of the existing unit tests. None observed, though. - Sebastian Kügler On Nov. 19, 2013, 9:58 a.m., Aleix Pol Gonzalez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/113920/ > ----------------------------------------------------------- > > (Updated Nov. 19, 2013, 9:58 a.m.) > > > Review request for Plasma. > > > Repository: plasma-framework > > > Description > ------- > > Stop using classes from kde4support: > - KComponentData is deprecated, it will be using > QCoreApplication::applicationName() and > QCoreApplication::applicationVersion(). Of course this means that the > existing shells will have to be ported. I have no problem with doing that > port myself, if I'm told where to look into. > - Drop usage of KLocale, ported to QLocale > - Drop usage of KStandardDirs, ported to QStandardPaths > - Drop usage of KIcon, ported to QIcon > > Furthermore, there's a module in src/declarativeimports/locale that IIUC > exposes KLocale bindings to QML. A specific plan to port it should happen as > well. > > > Diffs > ----- > > CMakeLists.txt a9da457 > src/declarativeimports/core/tests/CMakeLists.txt da0dbc5 > src/declarativeimports/core/tests/sortfiltermodeltest.cpp 93ffba4 > src/declarativeimports/locale/CMakeLists.txt 6aecb84 > src/declarativeimports/qtextracomponents/tests/CMakeLists.txt 2c64b7f > src/plasma/CMakeLists.txt c227a15 > src/plasma/private/dataenginemanager.cpp 1c235e0 > src/shell/CMakeLists.txt c19dbd2 > src/shell/activity.cpp 2523397 > src/shell/scripting/appinterface.cpp 63196f9 > src/shell/scripting/scriptengine.cpp 1250c20 > src/shell/shellcorona.cpp 2519e29 > > Diff: http://git.reviewboard.kde.org/r/113920/diff/ > > > Testing > ------- > > Builds, tests pass. > > I'm unsure what else to test. > > > Thanks, > > Aleix Pol Gonzalez > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel