On Wednesday 28 January 2015 19:30:40 Milian Wolff wrote: > On Wednesday 28 January 2015 18:18:51 Marko Käning wrote: > > Hi Milian, > > > > On 28 Jan 2015, at 16:59 , Milian Wolff <m...@milianw.de> wrote: > > > OK, can you tell me how to tweak cppcheckExtraArgs for individual > > > projects, > > > e.g. kdevplatform for now? > > > > for this project currently exists this file: > > --- > > $ cd PATH_TO_BUILD-KDE-ORG > > $ cat config/build/kdevplatform/kf5-qt5.cfg > > [DEFAULT] > > configureExtraArgs=-DBUILD_COVERAGE=ON > > > > [QualityCheck] > > runCppcheck=True > > > > --- > > which ensures that cppcheck is run at all. > > > > > > Now you simply add your additional arguments ‘-foo' and ‘-bar' to > > cppcheckExtraArgs like this: --- > > $ cat config/build/kdevplatform/kf5-qt5.cfg > > [DEFAULT] > > configureExtraArgs=-DBUILD_COVERAGE=ON > > > > [QualityCheck] > > runCppcheck=True > > cppcheckExtraArgs=-foo -bar > > > > --- > > and commit it back to the production branch of b-k-o [1]. > > Thanks! > > I only grepped for kdevplatform, and did not try to find anything that has > it in its name ;-) My bad! > > OK, so it's easy enough to add ignored paths, which I'll commit now. But I > wonder about the defines. Ben, why does the global config explicitly define > > -DKDE_IMPORT -DKDE_EXPORT > > in cppcheckCommand? By default, cppcheck would otherwise go over all defines > and ignore those that include #error - which would be exactly what I want > in kdevelop code. If instead -D is to be used, then we'd suddenly need to > add /all/ defines there. This triggers one error in KDevelop. > > Additionally, tons of files are ignored b/c the export macros are now > generated in the build folder, which is not added to the include paths. > Should we then add all include paths to cppcheck manually, and globally? Or > does someone know a better approach here? I only see the option to write a > custom script which gets the defines and include paths from the > CMake-generated compile_commands.json and then runs cppcheck on each target > with the configuration it gets from there. Still needs some hacks though to > work properly (i.e. I need to define __linux__ manually etc. pp., to > prevent errors in Qt headers that try to detect the system). > > For now, I'll just add more defines to the extra args, but I'd like to start > the discussion on how to handle this better in the future.
How do I sent review requests for the configuration on build.kde.org? https://paste.kde.org/pl6lf9j4g -- Milian Wolff m...@milianw.de http://milianw.de >> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<