gz-transport icon indicating copy to clipboard operation
gz-transport copied to clipboard

Wrong include headers for uuid on Ubuntu 18.04

Open osrf-migration opened this issue 4 years ago • 0 comments

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:

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

  1. 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.

osrf-migration avatar Sep 09 '19 20:09 osrf-migration