Details
-
Type:
Bug
-
Status: Done
-
Resolution: Done
-
Fix Version/s: None
-
Component/s: lsstDoxygen
-
Labels:None
-
Team:SQuaRE
Description
Observe the broken image in the Doxygen documentation for meas_base.
This image is present in the meas_base repository, and appears correctly when running scons doc directly. However, it doesn't make it on to the web version of the docs.
Attachments
Activity
Here's a tiny review for you. The last time I asked for a review of a change of just a few characters, I got tutted at – but since I'm not completely sure I understand how Doxygen is actually be invoked by Buildbot (based on the logs, it looks as though lsstDoxygen is cloned directly rather than being installed through eups, but I may be confused), I'd appreciate a sanity check if you have a moment.
Anyway, change is on tickets/DM-1752 in meas_base.
Sorry John, completely failed to notice this one in review.
The problem here is due to lsstDoxygen, which creates an "aggregate" Doxyfile by parsing and combining the Doxygen configurations from all the various packages. In order to do that properly (If we assume it can be done properly, but that's another story...), it has to canonicalize relative paths to things like example files and (in this case) images. Currently, it doesn't. The fix looks straightforward.