Fixed #13323 (building triage tool)

[danmar]

No description provided.

[firewave]

??? It is part of the CMake project. Just build the triage target. It is awkwardly conditional behind the BUILD_TESTS define though which I will change soon.

[danmar]

Well this will make it possible to build the triage tool directly and easily.

We can certainly keep the cmake building also and use that for the CI.

[danmar]

It is part of the CMake project.

I don't see a strong reason why it's part of the cmake project. This is only intended to be a developer tool not something users should see.

We could have a standalone cmake in triage folder that would be easier to build for us imho and for users they will have 1 fewer option when building cppcheck?

[firewave]

I don't see a strong reason why it's part of the cmake project. This is only intended to be a developer tool not something users should see.

Because that is our main build system. That's why it should be disabled by default (changes coming up).

We could have a standalone cmake in triage folder that would be easier to build for us imho and for users they will have 1 fewer option when building cppcheck?

That would increase the maintenance burden and add additional steps to the CI and the point is getting rid of unnecessary redundancies which detract from actually useful work.

If you really need it separate you can maintain it in a fork but it does not make sense to have it in the repo.

[firewave]

Well this will make it possible to build the triage tool directly and easily.

Then we need to revert everything related to qmake because we need to make sure it kept up-to-date and builds.