maliput / maliput_infrastructure

Driving simulation repository for workspace .repo indexes, Makefiles, etc.
BSD 3-Clause "New" or "Revised" License
0 stars 2 forks source link

Doxygen's namespace list doesn't show all the namespaces #235

Open francocipollone opened 3 years ago

francocipollone commented 3 years ago

Summary

This happens when the namespace doesn't have any class defined and only affects to the namespace list in the left panel.

Example:

image

In the left panel, we don't see the loader namespace however that namespace clearly exists as I am standing at the loader namespace image

Link to online doc

Known issue from doxygen

https://github.com/doxygen/doxygen/issues/8103

liangfok commented 3 years ago

Which version of Doxygen are we using? Looking at this comment, the issue should be fixed in version 1.9.0.

francocipollone commented 3 years ago

Which version of Doxygen are we using? Looking at this comment, the issue should be fixed in version 1.9.0.

Yes, I am checking this. We are using 1.8.13 (bionic) and 1.8.17 on focal. This is brought by ament_cmake_doxygen which installs doxygen via rosdep

liangfok commented 3 years ago

Okay, good to know that the fix is hopefully just to switch versions of Doxygen. Not urgent.

francocipollone commented 3 years ago

Based on this rule rosdep#base.yaml rosdep is basically calling apt install doxygen

So the corresponding ubuntu package is this one https://packages.ubuntu.com/search?keywords=doxygen: image

Therefore, given that the bug fix was released along with doxygen 1.9 version, sadly we can't be benefited from it unless we move to a newer Ubuntu version(21.04 at minimum).