-
Notifications
You must be signed in to change notification settings - Fork 6k
Windows build doc: add Graphviz to third party install list #28334
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
base: dev
Are you sure you want to change the base?
Conversation
|
That's a wrong fix, graphviz shouldn't be needed. |
|
Probably, but after install and rerun win.bat error message gone. |
|
It's still a wrong fix. It should build without graphviz installed. |
|
There is also the same issue: Error running win.bat #27899, it was resolved by Graphviz install. I agree that root cause could be something else. |
|
The right fix for #27899 is to remove doxygen so libheif doesn't look for graphviz |
|
Doxygen seems to be OFF in all occurrences https://github.com/search?q=repo%3Atelegramdesktop%2Ftdesktop%20doxygen&type=code |
I'm not sure what you want to say by that. The issue you linked says |
There is an error message
problems opening map file libheif/apidoc/html/heif_8h__incl.mapwhile runningwin.batwithout Graphviz installed.Add it to third party software list.