-
Notifications
You must be signed in to change notification settings - Fork 32
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
could only build with change in /usr/include/graphviz/types.h #33
Comments
which branch are you using? |
thanks for the reply. |
strange. |
hello, I guess sth went wrong with the include paths passed to the compiler when it happened:
I attach both the cmake log and the complete make log file. As written I managed to sucessfully build by changing the referred graphviz file, but of course that is wrong as it would break other graphviz based builds expecting graphviz devel files as they are ;) |
you're right. sorry. maybe we need to increase the minimum required graphviz version beyond 2.30.1 ? another possibility is to pass and yet another possibility is to add vcpkg support |
It looks to me like the As written, it will discover the include path for a KDStateMachineEditor/cmake/FindGraphviz.cmake Lines 76 to 80 in 2567f49
...But the Graphviz pkg-config data has the include path as $ pkg-config --cflags libcgraph
-I/usr/include/graphviz (This is with Graphviz 8.1.0 installed on Fedora 39, but I doubt it's changed anytime recently.) |
Looks to me like it should be, find_path(
GRAPHVIZ_INCLUDE_DIR
NAMES cgraph.h graph.h
HINTS ${_GRAPHVIZ_INCLUDE_DIR}
PATH_SUFFIXES graphviz
${_GRAPHVIZ_FIND_OPTS}
) (EDIT: Changed to look for |
hello,
I am pretty sure my fix is wrong but without changing
to
line 49 #include <graphviz/cgraph.h>
my build stopped with "cgraph.h not found".
I am using red hat rhel 7 with gcc 11.2.1 and graphviz-devel 2.30.1
Thanks, Martin.
The text was updated successfully, but these errors were encountered: