> On Feb. 8, 2015, 9:19 a.m., Laurent Montel wrote: > > about/kio_about.cpp, line 59 > > <https://git.reviewboard.kde.org/r/122471/diff/1/?file=347423#file347423line59> > > > > QApplication app(argc, argv); > > app.setApplicationName(QLatin1String("kio_about")); > > > > => you remove KComponentData which is in kdelibs4support
Well, QCoreApplication is enough. - David ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122471/#review75584 ----------------------------------------------------------- On Feb. 7, 2015, 10:23 p.m., David Narváez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/122471/ > ----------------------------------------------------------- > > (Updated Feb. 7, 2015, 10:23 p.m.) > > > Review request for kde-workspace and David Faure. > > > Repository: kio-extras > > > Description > ------- > > Basically porting away from QUrl and KDE_EXPORT > > > Diffs > ----- > > CMakeLists.txt 3379ce7 > about/kio_about.h 620d6aa > about/kio_about.cpp d7396d8 > > Diff: https://git.reviewboard.kde.org/r/122471/diff/ > > > Testing > ------- > > Tested compilation and installation: > > $ find /home/david/kio-install/ -name "*about*" > /home/david/kio-install/lib64/plugins/kio_about.so > > /home/david/kio-install/share/kservices5/about.protocol > > > Thanks, > > David Narváez > >