FindGraphviz.cmake uses DOT as well for finding the version of GraphViz that is installed -- not that the current module or code supports checking hthat version, but it's there.

In D8062#150767, @adridg wrote:

FindGraphviz.cmake uses DOT as well for finding the version of GraphViz that is installed -- not that the current module or code supports checking hthat version, but it's there.

Sorry, I should've mentioned that I have another branch here with a rewritten FindGraphviz.cmake that uses graphviz_version.h instead of dot for version checking. This review request is just testing the waters :-)

@jriddell @kossebau: ping?

I added kleag, the former author of kgraphviewer. Though he might still miss notifications from phabricator as before. Best contact him per email directly (find email on identity,kde,org)

In any case, instead of that manual warning, dot should be reported as dependency of TYPE RUNTIME, so it is listed in the cmake feature_summary. IIRC that needs a separate FindDot.cmake file, so there can be some

find_package(Dot QUIET)
set_package_properties(heaptrack_gui PROPERTIES
    DESCRIPTION "Tool which generates foo bar"
    URL "http://www.graphviz.org/"
    PURPOSE "Needed for foo bar"
    TYPE RUNTIME
)

Given you mentioned some further work on the cmake find files, that could go together as one.

@rakuco ping? Had you pursued your work here (incl. the rewritten FindGraphviz.cmake) or can we discard this review request as discontinued, to clean up our phab todo list?