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

ament_cmake_python: --symlink-install does not work correctly if node is renamed #562

Open
Jannkar opened this issue Nov 20, 2024 · 1 comment
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@Jannkar
Copy link

Jannkar commented Nov 20, 2024

I got an issue report on Turtle Nest ROS 2 package, notifying that --symlink-install option does not work correctly for packages that are built for C++ and Python, when a node is renamed.

Here is the full issue report: Jannkar/turtle_nest#15

I believe this might an issue on ament_cmake_python side?

Example of node renaming:

# Install Python executables
install(PROGRAMS
  ${PROJECT_NAME}/python_node.py
  DESTINATION lib/${PROJECT_NAME}
  RENAME python_node
)
@cottsay
Copy link
Contributor

cottsay commented Dec 12, 2024

This is indeed not a supported scenario:

As far as I can tell it isn't omitted for any particular reason. The pattern likely just wasn't popular when ament_cmake symlinking support was implemented. Feel free to submit a PR to add support for that option.

@cottsay cottsay added enhancement New feature or request help wanted Extra attention is needed labels Dec 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants