> On Nov. 20, 2014, 9 a.m., Luca Beltrame wrote: > > Actually it's still broken: I didn't notice because I did not build tests: > > > > > > /srv/jenkins/workspace/kde-baseapps_frameworks_qt5/konqueror/src/konqapplication.h:28:46: > > error: expected initializer before ‘:’ token > > > > The line being > > > > class KONQ_TESTS_EXPORT KonquerorApplication : public KApplication > > Luca Beltrame wrote: > And finally fixed.
I'm sorry, I'll pay more attention next time. I had to revert my tree to not include the other commits (of the other review requests I sent the same day) and had left it out when doing so. I'm not sure what happened regarding the KONQ_TESTS_EXPORT typo, since the build went ok (no clean build?) I don't have a linux build set up to double-check, so I rely on the review request. @Luca: Thanks for the prompt fix, and sorry again! - Andrius da Costa ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121078/#review70681 ----------------------------------------------------------- On Nov. 20, 2014, 3:29 a.m., Andrius da Costa Ribas wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/121078/ > ----------------------------------------------------------- > > (Updated Nov. 20, 2014, 3:29 a.m.) > > > Review request for KDE Base Apps, KDE Frameworks and kdewin. > > > Repository: kde-baseapps > > > Description > ------- > > Use CMake's generate_export_header whenever possible, and fix some > inconsistent exports. > Also, use Q_DECL_EXPORT consistently across occurrences of kdemain for > kdeinit apps (removing kdemacros.h where KDE_EXPORT was being in use). > > > Diffs > ----- > > dolphin/src/views/dolphinfileitemlistwidget.h 5f7c181 > dolphin/src/views/dolphinitemlistview.h 7fd72e5 > dolphin/src/views/versioncontrol/updateitemstatesthread.h 9b744d5 > dolphin/src/views/versioncontrol/versioncontrolobserver.h 034ef8b > dolphin/src/views/viewmodecontroller.h 9ea717a > dolphin/src/views/viewproperties.h d3ea428 > keditbookmarks/kbookmarkmodel/CMakeLists.txt 2f83af2 > keditbookmarks/kbookmarkmodel/kbookmarkmodel_export.h b759bf9 > keditbookmarks/main.cpp bc64193 > konqueror/client/kfmclient.cpp 276452b > konqueror/src/CMakeLists.txt 6831c88 > konqueror/src/konq_statusbarmessagelabel.h a115d39 > konqueror/src/konqmain.cpp 7a73231 > konqueror/src/konqprivate_export.h 2dec78b > dolphin/src/libdolphin_export.h ff0f125 > dolphin/src/main.cpp 91d32a0 > dolphin/src/kitemviews/private/kitemlistviewlayouter.h 0efcab1 > dolphin/src/CMakeLists.txt c074908 > dolphin/src/kitemviews/kfileitemlistview.h 9f07c9d > dolphin/src/kitemviews/kfileitemlistwidget.h b7061d3 > dolphin/src/kitemviews/kfileitemmodel.h 535e0f3 > dolphin/src/kitemviews/kfileitemmodelrolesupdater.h e8b186a > dolphin/src/kitemviews/kitemlistcontainer.h 55a9a11 > dolphin/src/kitemviews/kitemlistcontroller.h 99df531 > dolphin/src/kitemviews/kitemlistgroupheader.h 66fe929 > dolphin/src/kitemviews/kitemlistheader.h fe2d0ac > dolphin/src/kitemviews/kitemlistselectionmanager.h c4decd3 > dolphin/src/kitemviews/kitemliststyleoption.h 782dd0e > dolphin/src/kitemviews/kitemlistview.h cc0754f > dolphin/src/kitemviews/kitemlistviewaccessible.h 674f1d1 > dolphin/src/kitemviews/kitemlistwidget.h 07fc789 > dolphin/src/kitemviews/kitemmodelbase.h 283cfa5 > dolphin/src/kitemviews/kstandarditem.h 59190bf > dolphin/src/kitemviews/kstandarditemlistgroupheader.h 871c93d > dolphin/src/kitemviews/kstandarditemlistview.h eb07c7f > dolphin/src/kitemviews/kstandarditemlistwidget.h 7683f84 > dolphin/src/kitemviews/kstandarditemmodel.h 9d1db1a > dolphin/src/kitemviews/private/kbaloorolesprovider.h f1ad5c7 > dolphin/src/kitemviews/private/kfileitemmodeldirlister.h 94a0e39 > dolphin/src/kitemviews/private/kfileitemmodelfilter.h e4fb0a2 > dolphin/src/kitemviews/private/kitemlistheaderwidget.h b99f45f > dolphin/src/kitemviews/private/kitemlistkeyboardsearchmanager.h 661178d > dolphin/src/kitemviews/private/kitemlistrubberband.h aea58af > dolphin/src/kitemviews/private/kitemlistselectiontoggle.h 8a1a857 > dolphin/src/kitemviews/private/kitemlistsizehintresolver.h a0ad033 > dolphin/src/kitemviews/private/kitemlistsmoothscroller.h ea56822 > dolphin/src/kitemviews/private/kitemlistviewanimation.h e2e5338 > dolphin/src/kitemviews/private/kpixmapmodifier.h 4f863c3 > > Diff: https://git.reviewboard.kde.org/r/121078/diff/ > > > Testing > ------- > > It builds (MSVC2013 - 64bit) after this patch (along other patches I'm > sending to review today). Kdebase-apps is still not very functional, though > (missing icons and weird UI). > > > Thanks, > > Andrius da Costa Ribas > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel