Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add /usr/lib/<pkg> and /usr/libexec/<pkg> to search paths #467

Open
wants to merge 1 commit into
base: rolling
Choose a base branch
from

Conversation

sloretz
Copy link
Contributor

@sloretz sloretz commented Mar 19, 2020

This is a proposal following ros2/ci#400 (comment). This PR adds <ws>/usr/lib/<pkg> and <ws>/usr/libexec/<pkg> to the search paths used by ros2 run to comply with the Linux Filesystem Hierarchy Standard. It looks like the current practice of <ws>/lib/<pkg> doesn't match, but I left that in to make this be an enhancement rather than a disruptive change.

@sloretz sloretz added the enhancement New feature or request label Mar 19, 2020
@sloretz sloretz self-assigned this Mar 19, 2020
Copy link
Member

@dirk-thomas dirk-thomas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The current implementation follows REP 122. Until that REP has been iterated on the implementation shouldn't change.

@audrow
Copy link
Member

audrow commented Mar 12, 2021

Hey @sloretz, are you still interested in getting this feature in? Perhaps you or I can make PR with a small addition to REP 122.

@audrow audrow changed the base branch from master to rolling June 28, 2022 14:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants