-
Notifications
You must be signed in to change notification settings - Fork 37
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
Dox fix #244
Dox fix #244
Conversation
Very nice thank you. To avoid overwriting the EDM4hep/scripts/updateReadmeLinks.py Line 24 in 67bdd9a
and here EDM4hep/scripts/updateReadmeLinks.py Line 35 in 67bdd9a
|
undo master/main change to link to podio
Done |
# copy images needed for documentation to build folder and install them | ||
configure_file(${CMAKE_CURRENT_SOURCE_DIR}/doc/edm4hep_diagram.svg | ||
${CMAKE_CURRENT_BINARY_DIR}/doxygen/html/doc/edm4hep_diagram.svg COPYONLY) | ||
install(FILES ${CMAKE_CURRENT_BINARY_DIR}/doxygen/html/doc/edm4hep_diagram.svg | ||
DESTINATION ${CMAKE_INSTALL_DOCDIR}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This does not make the edm4hep diagram disappear on the main page?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe not because it uses the README which has a link to it but I haven't tested it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doxygen copies the file on its own when generating html from md. Some older version perhaps required manual intervention
$ find build -name edm4hep_diagram.svg
build/doxygen/html/edm4hep_diagram.svg
$ find InstallArea/ -name edm4hep_diagram.svg
InstallArea/share/doc/EDM4HEP/doxygen/doxygen/html/edm4hep_diagram.svg
BEGINRELEASENOTES
ENDRELEASENOTES