gz-transport
gz-transport copied to clipboard
Wrong include headers for uuid on Ubuntu 18.04
Original report (archived issue) by Silvio Traversaro (Bitbucket: traversaro).
Prerequisites
-
[x] Put an X between the brackets on this line if you have done all of the following:
- Checked the Q&A board for common solutions: http://answers.gazebosim.org
- Checked that your issue isn't already filed.
- Checked that there is not already an Ignition package that provides the described functionality: https://ignitionrobotics.org/libs
Description
On Ubuntu 18.04, ignition-transport4 includes the uuid/uuid.h
, but it should include directly the uuid.h
file as the UUID include directories are <prefix>/include/uuid
, not <prefix>/include
.
Steps to Reproduce
- Inspect the
UUID_INCLUDE_DIRS
CMake variable.
Expected behavior:
It should contain a directory that contains a uuid/uuid.h
file, that is then included in https://github.com/ignitionrobotics/ign-transport/blob/a9318bdb9a233e32a27bbe71f99d534b23315b7a/include/ignition/transport/Uuid.hh#L32 .
Actual behavior:
The directory contains a uuid.h
file .
Reproduces how often:
All the time.
Versions
Ubuntu 18.04, ignition-transport 4.
Additional Information
The problem does not emerge because uuid is tipically installed either in the system prefix /usr
or it shares the prefix with other dependencies, and so the <prefix>/include
directory is added to the include paths in any case. However it could create problems if uuid
is installed in a separated prefix.