scpeters-test / sdformat

Simulation Description Format (SDF) parser and description files.
http://sdformat.org
Other
1 stars 0 forks source link

sdformat install not relocatable: sdf::findFile searches original install location #198

Open scpeters-test opened 6 years ago

scpeters-test commented 6 years ago

Original report (archived issue) by Steve Peters (Bitbucket: Steven Peters).


As reported here: https://github.com/osrf/homebrew-simulation/issues/479

The issue is that the install path searched by sdf::findFile is the install location at compile time:

We would need to change the logic of sdf::findFile to make it fully relocatable.

scpeters-test commented 6 years ago

Original comment by Jeongseok Lee (Bitbucket: jlee02, GitHub: jslee02).


DART also had the same issue and adopted the workaround of using the environment variable, which I believe sdformat already does. One possible solution I can think of now is searching typical install locations (like find_package does), which are platform dependent.

scpeters-test commented 6 years ago

Original comment by Steve Peters (Bitbucket: Steven Peters).


Yes, SDF already checks the SDF_PATH environment variable:

but it does take an extra step to set that.

scpeters-test commented 6 years ago

Original comment by Silvio Traversaro (Bitbucket: traversaro).


Relevant issue (I was not finding it anymore, so I think it is useful to link it here): https://scpeters-test.github.io/gh-pages_sdformat_fast_export/#!/osrf/sdformat/issues/141/embed-sdf-xml-files-into-library (#141) .