cfg/kde.cfg - add more KDE macros that are widely used

[winterz]

No description provided.

[chrchr-github]

Thanks for your contribution. Can you add some test code in test/cfg/kde.cpp?

[winterz]

Thanks for your contribution. Can you add some test code in test/cfg/kde.cpp?

ok.
Am I permitted to use Qt constructs in the kde.cpp file? i.e. require '--library=qt'

[winterz]

test program updated. it does require Qt though

[chrchr-github]

Probably --library=qt is required here: https://github.com/danmar/cppcheck/blob/6d61fb60f119ff192c56bd49e96e97bc6f6f7ed4/test/cfg/runtests.sh#L510C13-L510C77

[winterz]

Probably --library=qt is required here: https://github.com/danmar/cppcheck/blob/6d61fb60f119ff192c56bd49e96e97bc6f6f7ed4/test/cfg/runtests.sh#L510C13-L510C77

my last push includes that yet the CI is still failing.
I'm not sure what's missing

[chrchr-github]

my last push includes that yet the CI is still failing. I'm not sure what's missing

I must have missed that. Maybe omit the quotes around "qt"?

[chrchr-github]

@firewave Do you have any idea?

[firewave]

You also need to adjust test/CMakeLists,txt.

[winterz]

thanks @firewave , adjusting test/CMakeLists.txt seems to have worked

[firewave]

thanks @firewave , adjusting test/CMakeLists.txt seems to have worked

It has to, because that code is the (basic) CMake implementation of runtest.sh. They have to be kept in sync. Before anyone asks why we have two. The CMake one is more basic and portable, and the Shell one has more (non-portable) functionality.

[winterz]

great. can someone with write permission merge this please?