Skip to content

Conversation

@vovodroid
Copy link

There is an error message problems opening map file libheif/apidoc/html/heif_8h__incl.map while running win.bat without Graphviz installed.

Add it to third party software list.

@CLAassistant
Copy link

CLAassistant commented Aug 30, 2024

CLA assistant check
All committers have signed the CLA.

@ilya-fedin
Copy link
Contributor

That's a wrong fix, graphviz shouldn't be needed.

@vovodroid
Copy link
Author

Probably, but after install and rerun win.bat error message gone.

@ilya-fedin
Copy link
Contributor

It's still a wrong fix. It should build without graphviz installed.

@vovodroid
Copy link
Author

vovodroid commented Aug 31, 2024

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.

@ilya-fedin
Copy link
Contributor

ilya-fedin commented Aug 31, 2024

The right fix for #27899 is to remove doxygen so libheif doesn't look for graphviz

@vovodroid
Copy link
Author

Doxygen seems to be OFF in all occurrences

https://github.com/search?q=repo%3Atelegramdesktop%2Ftdesktop%20doxygen&type=code

@ilya-fedin
Copy link
Contributor

ilya-fedin commented Aug 31, 2024

Doxygen seems to be OFF in all occurrences

I'm not sure what you want to say by that. The issue you linked says

CUSTOMBUILD : error : problems opening map file C:/TBuild/Libraries/win64/libheif/apidoc/html/heif_8h__incl.map for inc
lusion in the docs! [C:\TBuild\Libraries\win64\libheif\doc_doxygen.vcxproj]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants