RFC/call for assistance: Merging kdev-clang-tidy into kdevelop repo
Friedrich W. H. Kossebau
kossebau at kde.org
Thu Oct 4 13:27:34 BST 2018
Hi,
I would like to propose the merge of the Clang-Tidy integration plugin, kdev-
clang-tidy, into the kdevelop repo (master branch), as soon as it has been
decided about. To join the static analyzer buddies kdev-clazy & kdev-cppcheck.
State:
* the current code of the plugin has seen some release already this year
* works for me (tm)
* builds clean on KDE CI
https://build.kde.org/job/KDevelop/job/kdev-clang-tidy
* no bugs reported (but also no other users yet known :) )
https://bugs.kde.org/buglist.cgi?component=Analyzer%3A%20Clang-Tidy&product=kdevelop
Get some more information about the plugin (incl. screenshots) on
https://frinring.wordpress.com/2018/08/30/improve-your-cpp-code-in-kdevelop-with-clang-tidy/
ARE WE OKAY WITH THAT?
So far it seems KDevelop contributors/maintainers are positive about such a
merge. But let's make sure and get an explicit Yay/Nay here :)
EXECUTION
Given kdev-clang-tidy plugin code has seen multiple authors, a simple commit
with a dump of the current working copy might lose too much info.
So semms best the git history of the kdev-clang-tidy gets completely merged?
Anyone experienced with that, who could tell what to do or do it themselves?
I will do what I can do, but need a helping hand with that nerge bit.
BUT WHAT ABOUT THE PLANS TO DO MORE INDEPENDENT RELEASES?
Doing an Autumn reevaluation of my maintained projects, I found the my old
plan to continue independent development of kdev-clang-tidy with new feature
releases parallel to stable KDevelop 5.3 and then merging in time for KDevelop
5.4 lacks my motivation now, as the additional planned features are currently
not needed by myself and I also never got any feedback on the plugin.
Given I use KDevelop from master myself, the source location of kdev-clang-
tidy makes no difference for me :)
Actually, having kdev-clang-tidy in master sooner might even help with fixing
my current biggest bummer, having to manually enable
CMAKE_EXPORT_COMPILE_COMMANDS with lots of steps, when that could be done
code-wise (patch started somewhere in a branch).
Cheers
Friedrich
More information about the KDevelop-devel
mailing list