Re: Review Request 112556: Move itemviews to tier2
On Tuesday 10 September 2013 21:46:45 Christoph Feck wrote: So what about renaming itemmodels/itemviews to kitemmodels/kitemviews, or is it too late for that? I'd be fine with that. There are indeed other item models and item views than the ones provided by KF5, so a k makes sense. Stephen, what do you think? Do we plan to have a proper/final let's look at what we did API review after everything has settled? That would be good indeed, but let's do it along the way as well rather than postpone it all :) -- David Faure, fa...@kde.org, http://www.davidfaure.fr Working on KDE, in particular KDE Frameworks 5 ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 112556: Move itemviews to tier2
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112556/ --- (Updated Sept. 13, 2013, 5:02 p.m.) Status -- This change has been discarded. Review request for KDE Frameworks. Description --- In itemviews/CMakeLists.txt: I took the find_package(Qt5) out of the if (NOT kdelibs_SOURCE_DIR) test otherwise it would not build within kdelibs. Diffs - staging/CMakeLists.txt f0b6bf0 staging/itemviews/CMakeLists.txt f5c3c8d staging/itemviews/ItemViewsConfig.cmake.in c10f49f staging/itemviews/autotests/CMakeLists.txt staging/itemviews/autotests/klistwidgetsearchlinetest.cpp staging/itemviews/src/CMakeLists.txt staging/itemviews/src/kcategorizedsortfilterproxymodel.h staging/itemviews/src/kcategorizedsortfilterproxymodel.cpp staging/itemviews/src/kcategorizedsortfilterproxymodel_p.h staging/itemviews/src/kcategorizedview.h staging/itemviews/src/kcategorizedview.cpp staging/itemviews/src/kcategorizedview_p.h staging/itemviews/src/kcategorydrawer.h staging/itemviews/src/kcategorydrawer.cpp staging/itemviews/src/kextendableitemdelegate.h staging/itemviews/src/kextendableitemdelegate.cpp staging/itemviews/src/kfilterproxysearchline.h staging/itemviews/src/kfilterproxysearchline.cpp staging/itemviews/src/klistwidgetsearchline.h staging/itemviews/src/klistwidgetsearchline.cpp staging/itemviews/src/ktreewidgetsearchline.h staging/itemviews/src/ktreewidgetsearchline.cpp staging/itemviews/src/kwidgetitemdelegate.h staging/itemviews/src/kwidgetitemdelegate.cpp staging/itemviews/src/kwidgetitemdelegate_p.h staging/itemviews/src/kwidgetitemdelegatepool.cpp staging/itemviews/src/kwidgetitemdelegatepool_p.h staging/itemviews/tests/CMakeLists.txt staging/itemviews/tests/kcategorizedviewtest.cpp staging/itemviews/tests/ktreewidgetsearchlinetest.h staging/itemviews/tests/ktreewidgetsearchlinetest.cpp staging/itemviews/tests/kwidgetitemdelegatetest.cpp tier2/CMakeLists.txt 62faacb Diff: http://git.reviewboard.kde.org/r/112556/diff/ Testing --- Builds and passes tests when build from kdelibs and standalone. Thanks, Aurélien Gâteau ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 112556: Move itemviews to tier2
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112556/ --- (Updated Sept. 11, 2013, 3:22 p.m.) Review request for KDE Frameworks. Changes --- Make toplevel CMakeLists.txt and ItemViewConfig.cmake.in look more like other frameworks. Description --- In itemviews/CMakeLists.txt: I took the find_package(Qt5) out of the if (NOT kdelibs_SOURCE_DIR) test otherwise it would not build within kdelibs. Diffs (updated) - staging/CMakeLists.txt f0b6bf0 staging/itemviews/CMakeLists.txt f5c3c8d staging/itemviews/ItemViewsConfig.cmake.in c10f49f staging/itemviews/autotests/CMakeLists.txt staging/itemviews/autotests/klistwidgetsearchlinetest.cpp staging/itemviews/src/CMakeLists.txt staging/itemviews/src/kcategorizedsortfilterproxymodel.h staging/itemviews/src/kcategorizedsortfilterproxymodel.cpp staging/itemviews/src/kcategorizedsortfilterproxymodel_p.h staging/itemviews/src/kcategorizedview.h staging/itemviews/src/kcategorizedview.cpp staging/itemviews/src/kcategorizedview_p.h staging/itemviews/src/kcategorydrawer.h staging/itemviews/src/kcategorydrawer.cpp staging/itemviews/src/kextendableitemdelegate.h staging/itemviews/src/kextendableitemdelegate.cpp staging/itemviews/src/kfilterproxysearchline.h staging/itemviews/src/kfilterproxysearchline.cpp staging/itemviews/src/klistwidgetsearchline.h staging/itemviews/src/klistwidgetsearchline.cpp staging/itemviews/src/ktreewidgetsearchline.h staging/itemviews/src/ktreewidgetsearchline.cpp staging/itemviews/src/kwidgetitemdelegate.h staging/itemviews/src/kwidgetitemdelegate.cpp staging/itemviews/src/kwidgetitemdelegate_p.h staging/itemviews/src/kwidgetitemdelegatepool.cpp staging/itemviews/src/kwidgetitemdelegatepool_p.h staging/itemviews/tests/CMakeLists.txt staging/itemviews/tests/kcategorizedviewtest.cpp staging/itemviews/tests/ktreewidgetsearchlinetest.h staging/itemviews/tests/ktreewidgetsearchlinetest.cpp staging/itemviews/tests/kwidgetitemdelegatetest.cpp tier2/CMakeLists.txt 62faacb Diff: http://git.reviewboard.kde.org/r/112556/diff/ Testing --- Builds and passes tests when build from kdelibs and standalone. Thanks, Aurélien Gâteau ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 112556: Move itemviews to tier2
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112556/ --- (Updated Sept. 10, 2013, 12:45 p.m.) Review request for KDE Frameworks. Changes --- Regenerated diff using --find-copies-harder Description --- In itemviews/CMakeLists.txt: I took the find_package(Qt5) out of the if (NOT kdelibs_SOURCE_DIR) test otherwise it would not build within kdelibs. Diffs (updated) - staging/CMakeLists.txt f0b6bf0 staging/itemviews/CMakeLists.txt f5c3c8d staging/itemviews/ItemViewsConfig.cmake.in staging/itemviews/autotests/CMakeLists.txt staging/itemviews/autotests/klistwidgetsearchlinetest.cpp staging/itemviews/src/CMakeLists.txt staging/itemviews/src/kcategorizedsortfilterproxymodel.h staging/itemviews/src/kcategorizedsortfilterproxymodel.cpp staging/itemviews/src/kcategorizedsortfilterproxymodel_p.h staging/itemviews/src/kcategorizedview.h staging/itemviews/src/kcategorizedview.cpp staging/itemviews/src/kcategorizedview_p.h staging/itemviews/src/kcategorydrawer.h staging/itemviews/src/kcategorydrawer.cpp staging/itemviews/src/kextendableitemdelegate.h staging/itemviews/src/kextendableitemdelegate.cpp staging/itemviews/src/kfilterproxysearchline.h staging/itemviews/src/kfilterproxysearchline.cpp staging/itemviews/src/klistwidgetsearchline.h staging/itemviews/src/klistwidgetsearchline.cpp staging/itemviews/src/ktreewidgetsearchline.h staging/itemviews/src/ktreewidgetsearchline.cpp staging/itemviews/src/kwidgetitemdelegate.h staging/itemviews/src/kwidgetitemdelegate.cpp staging/itemviews/src/kwidgetitemdelegate_p.h staging/itemviews/src/kwidgetitemdelegatepool.cpp staging/itemviews/src/kwidgetitemdelegatepool_p.h staging/itemviews/tests/CMakeLists.txt staging/itemviews/tests/kcategorizedviewtest.cpp staging/itemviews/tests/ktreewidgetsearchlinetest.h staging/itemviews/tests/ktreewidgetsearchlinetest.cpp staging/itemviews/tests/kwidgetitemdelegatetest.cpp tier2/CMakeLists.txt 62faacb Diff: http://git.reviewboard.kde.org/r/112556/diff/ Testing --- Builds and passes tests when build from kdelibs and standalone. Thanks, Aurélien Gâteau ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 112556: Move itemviews to tier2
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112556/#review39773 --- Ship it! Ship It! - David Faure On Sept. 10, 2013, 10:45 a.m., Aurélien Gâteau wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112556/ --- (Updated Sept. 10, 2013, 10:45 a.m.) Review request for KDE Frameworks. Description --- In itemviews/CMakeLists.txt: I took the find_package(Qt5) out of the if (NOT kdelibs_SOURCE_DIR) test otherwise it would not build within kdelibs. Diffs - staging/CMakeLists.txt f0b6bf0 staging/itemviews/CMakeLists.txt f5c3c8d staging/itemviews/ItemViewsConfig.cmake.in staging/itemviews/autotests/CMakeLists.txt staging/itemviews/autotests/klistwidgetsearchlinetest.cpp staging/itemviews/src/CMakeLists.txt staging/itemviews/src/kcategorizedsortfilterproxymodel.h staging/itemviews/src/kcategorizedsortfilterproxymodel.cpp staging/itemviews/src/kcategorizedsortfilterproxymodel_p.h staging/itemviews/src/kcategorizedview.h staging/itemviews/src/kcategorizedview.cpp staging/itemviews/src/kcategorizedview_p.h staging/itemviews/src/kcategorydrawer.h staging/itemviews/src/kcategorydrawer.cpp staging/itemviews/src/kextendableitemdelegate.h staging/itemviews/src/kextendableitemdelegate.cpp staging/itemviews/src/kfilterproxysearchline.h staging/itemviews/src/kfilterproxysearchline.cpp staging/itemviews/src/klistwidgetsearchline.h staging/itemviews/src/klistwidgetsearchline.cpp staging/itemviews/src/ktreewidgetsearchline.h staging/itemviews/src/ktreewidgetsearchline.cpp staging/itemviews/src/kwidgetitemdelegate.h staging/itemviews/src/kwidgetitemdelegate.cpp staging/itemviews/src/kwidgetitemdelegate_p.h staging/itemviews/src/kwidgetitemdelegatepool.cpp staging/itemviews/src/kwidgetitemdelegatepool_p.h staging/itemviews/tests/CMakeLists.txt staging/itemviews/tests/kcategorizedviewtest.cpp staging/itemviews/tests/ktreewidgetsearchlinetest.h staging/itemviews/tests/ktreewidgetsearchlinetest.cpp staging/itemviews/tests/kwidgetitemdelegatetest.cpp tier2/CMakeLists.txt 62faacb Diff: http://git.reviewboard.kde.org/r/112556/diff/ Testing --- Builds and passes tests when build from kdelibs and standalone. Thanks, Aurélien Gâteau ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 112556: Move itemviews to tier2
On Sept. 10, 2013, 9:06 p.m., David Faure wrote: Ship It! So what about renaming itemmodels/itemviews to kitemmodels/kitemviews, or is it too late for that? Do we plan to have a proper/final let's look at what we did API review after everything has settled? - Christoph --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112556/#review39773 --- On Sept. 10, 2013, 10:45 a.m., Aurélien Gâteau wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112556/ --- (Updated Sept. 10, 2013, 10:45 a.m.) Review request for KDE Frameworks. Description --- In itemviews/CMakeLists.txt: I took the find_package(Qt5) out of the if (NOT kdelibs_SOURCE_DIR) test otherwise it would not build within kdelibs. Diffs - staging/CMakeLists.txt f0b6bf0 staging/itemviews/CMakeLists.txt f5c3c8d staging/itemviews/ItemViewsConfig.cmake.in staging/itemviews/autotests/CMakeLists.txt staging/itemviews/autotests/klistwidgetsearchlinetest.cpp staging/itemviews/src/CMakeLists.txt staging/itemviews/src/kcategorizedsortfilterproxymodel.h staging/itemviews/src/kcategorizedsortfilterproxymodel.cpp staging/itemviews/src/kcategorizedsortfilterproxymodel_p.h staging/itemviews/src/kcategorizedview.h staging/itemviews/src/kcategorizedview.cpp staging/itemviews/src/kcategorizedview_p.h staging/itemviews/src/kcategorydrawer.h staging/itemviews/src/kcategorydrawer.cpp staging/itemviews/src/kextendableitemdelegate.h staging/itemviews/src/kextendableitemdelegate.cpp staging/itemviews/src/kfilterproxysearchline.h staging/itemviews/src/kfilterproxysearchline.cpp staging/itemviews/src/klistwidgetsearchline.h staging/itemviews/src/klistwidgetsearchline.cpp staging/itemviews/src/ktreewidgetsearchline.h staging/itemviews/src/ktreewidgetsearchline.cpp staging/itemviews/src/kwidgetitemdelegate.h staging/itemviews/src/kwidgetitemdelegate.cpp staging/itemviews/src/kwidgetitemdelegate_p.h staging/itemviews/src/kwidgetitemdelegatepool.cpp staging/itemviews/src/kwidgetitemdelegatepool_p.h staging/itemviews/tests/CMakeLists.txt staging/itemviews/tests/kcategorizedviewtest.cpp staging/itemviews/tests/ktreewidgetsearchlinetest.h staging/itemviews/tests/ktreewidgetsearchlinetest.cpp staging/itemviews/tests/kwidgetitemdelegatetest.cpp tier2/CMakeLists.txt 62faacb Diff: http://git.reviewboard.kde.org/r/112556/diff/ Testing --- Builds and passes tests when build from kdelibs and standalone. Thanks, Aurélien Gâteau ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 112556: Move itemviews to tier2
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112556/ --- (Updated Sept. 9, 2013, 8:44 a.m.) Review request for KDE Frameworks. Changes --- Assign to correct group Description --- In itemviews/CMakeLists.txt: I took the find_package(Qt5) out of the if (NOT kdelibs_SOURCE_DIR) test otherwise it would not build within kdelibs. Diffs - staging/CMakeLists.txt f0b6bf0 staging/itemviews/CMakeLists.txt f5c3c8d staging/itemviews/ItemViewsConfig.cmake.in c10f49f staging/itemviews/autotests/CMakeLists.txt 305a6d1 staging/itemviews/autotests/klistwidgetsearchlinetest.cpp 37bcb5e staging/itemviews/src/CMakeLists.txt 353a413 staging/itemviews/src/kcategorizedsortfilterproxymodel.h a21e7ca staging/itemviews/src/kcategorizedsortfilterproxymodel.cpp c8b652d staging/itemviews/src/kcategorizedsortfilterproxymodel_p.h eb1a67b staging/itemviews/src/kcategorizedview.h c8035c5 staging/itemviews/src/kcategorizedview.cpp 35fafae staging/itemviews/src/kcategorizedview_p.h 4c175fb staging/itemviews/src/kcategorydrawer.h 3665457 staging/itemviews/src/kcategorydrawer.cpp 0fb857e staging/itemviews/src/kextendableitemdelegate.h 386730d staging/itemviews/src/kextendableitemdelegate.cpp f7515ce staging/itemviews/src/kfilterproxysearchline.h 85059d7 staging/itemviews/src/kfilterproxysearchline.cpp c8ccf89 staging/itemviews/src/klistwidgetsearchline.h f38dc83 staging/itemviews/src/klistwidgetsearchline.cpp 2961df9 staging/itemviews/src/ktreewidgetsearchline.h 3570080 staging/itemviews/src/ktreewidgetsearchline.cpp 050bf27 staging/itemviews/src/kwidgetitemdelegate.h f59dc10 staging/itemviews/src/kwidgetitemdelegate.cpp ff7bdb1 staging/itemviews/src/kwidgetitemdelegate_p.h 896fc2d staging/itemviews/src/kwidgetitemdelegatepool.cpp c4dd714 staging/itemviews/src/kwidgetitemdelegatepool_p.h f632d40 staging/itemviews/tests/CMakeLists.txt 32b0158 staging/itemviews/tests/kcategorizedviewtest.cpp 116278c staging/itemviews/tests/ktreewidgetsearchlinetest.h 77fd758 staging/itemviews/tests/ktreewidgetsearchlinetest.cpp 74992ee staging/itemviews/tests/kwidgetitemdelegatetest.cpp cfca1f9 tier2/CMakeLists.txt 62faacb tier2/itemviews/CMakeLists.txt PRE-CREATION tier2/itemviews/ItemViewsConfig.cmake.in PRE-CREATION tier2/itemviews/autotests/CMakeLists.txt PRE-CREATION tier2/itemviews/autotests/klistwidgetsearchlinetest.cpp PRE-CREATION tier2/itemviews/src/CMakeLists.txt PRE-CREATION tier2/itemviews/src/kcategorizedsortfilterproxymodel.h PRE-CREATION tier2/itemviews/src/kcategorizedsortfilterproxymodel.cpp PRE-CREATION tier2/itemviews/src/kcategorizedsortfilterproxymodel_p.h PRE-CREATION tier2/itemviews/src/kcategorizedview.h PRE-CREATION tier2/itemviews/src/kcategorizedview.cpp PRE-CREATION tier2/itemviews/src/kcategorizedview_p.h PRE-CREATION tier2/itemviews/src/kcategorydrawer.h PRE-CREATION tier2/itemviews/src/kcategorydrawer.cpp PRE-CREATION tier2/itemviews/src/kextendableitemdelegate.h PRE-CREATION tier2/itemviews/src/kextendableitemdelegate.cpp PRE-CREATION tier2/itemviews/src/kfilterproxysearchline.h PRE-CREATION tier2/itemviews/src/kfilterproxysearchline.cpp PRE-CREATION tier2/itemviews/src/klistwidgetsearchline.h PRE-CREATION tier2/itemviews/src/klistwidgetsearchline.cpp PRE-CREATION tier2/itemviews/src/ktreewidgetsearchline.h PRE-CREATION tier2/itemviews/src/ktreewidgetsearchline.cpp PRE-CREATION tier2/itemviews/src/kwidgetitemdelegate.h PRE-CREATION tier2/itemviews/src/kwidgetitemdelegate.cpp PRE-CREATION tier2/itemviews/src/kwidgetitemdelegate_p.h PRE-CREATION tier2/itemviews/src/kwidgetitemdelegatepool.cpp PRE-CREATION tier2/itemviews/src/kwidgetitemdelegatepool_p.h PRE-CREATION tier2/itemviews/tests/CMakeLists.txt PRE-CREATION tier2/itemviews/tests/kcategorizedviewtest.cpp PRE-CREATION tier2/itemviews/tests/ktreewidgetsearchlinetest.h PRE-CREATION tier2/itemviews/tests/ktreewidgetsearchlinetest.cpp PRE-CREATION tier2/itemviews/tests/kwidgetitemdelegatetest.cpp PRE-CREATION Diff: http://git.reviewboard.kde.org/r/112556/diff/ Testing --- Builds and passes tests when build from kdelibs and standalone. Thanks, Aurélien Gâteau ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel