Skip to content

Change cp command to use -rfL option#335

Merged
Amronos merged 1 commit into
micro-ROS:rollingfrom
Caedael:rolling
May 13, 2026
Merged

Change cp command to use -rfL option#335
Amronos merged 1 commit into
micro-ROS:rollingfrom
Caedael:rolling

Conversation

@Caedael
Copy link
Copy Markdown
Contributor

@Caedael Caedael commented Apr 30, 2026

resolve symlinks from extra_packages folder

#320
(changed branch to rolling)

resolve symlinks from extra_packages folder
Copy link
Copy Markdown
Member

@Amronos Amronos left a comment

Choose a reason for hiding this comment

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

Thanks! The CI failures are unrelated to this PR and will be fixed with #323.

@Amronos Amronos requested a review from Narukara April 30, 2026 14:13
@Narukara
Copy link
Copy Markdown
Contributor

Narukara commented May 6, 2026

May I ask — did you actually encounter a problem that required cp -rfL to fix? 🤔

@Caedael
Copy link
Copy Markdown
Contributor Author

Caedael commented May 11, 2026

May I ask — did you actually encounter a problem that required cp -rfL to fix? 🤔

When i work with custom messages, i don't wanted to copy them manually every time after a change from my workspace to the extra packages folder.

@Narukara
Copy link
Copy Markdown
Contributor

When i work with custom messages, i don't wanted to copy them manually every time after a change from my workspace to the extra packages folder.

I'm sorry, but I didn't understand what you meant. Could you please explain in more detail?

@Amronos, do you have any thoughts on this?

@Amronos
Copy link
Copy Markdown
Member

Amronos commented May 12, 2026

From what I understand, -L properly copies the directories that the symlinks point to.
It is needed as some people may symlink their own packages into the extra_packages directory to iterate faster on those packages.

@Narukara
Copy link
Copy Markdown
Contributor

From what I understand, -L properly copies the directories that the symlinks point to. It is needed as some people may symlink their own packages into the extra_packages directory to iterate faster on those packages.

Oh, I forgot that symbolic links can point to entire directories. Thanks for the clarification!

@Amronos Amronos merged commit 2f73852 into micro-ROS:rolling May 13, 2026
2 of 7 checks passed
Amronos pushed a commit that referenced this pull request May 13, 2026
Resolve symlinks from the `extra_packages` directory

(cherry picked from commit 2f73852)

Co-authored-by: Adrian <67166322+Caedael@users.noreply.github.com>
Amronos pushed a commit that referenced this pull request May 13, 2026
Resolve symlinks from the `extra_packages` directory

(cherry picked from commit 2f73852)

Co-authored-by: Adrian <67166322+Caedael@users.noreply.github.com>
Amronos added a commit that referenced this pull request May 13, 2026
Resolve symlinks from the `extra_packages` directory

(cherry picked from commit 2f73852)

Co-authored-by: Adrian <67166322+Caedael@users.noreply.github.com>
Co-authored-by: Aarav Gupta <aarav@spikonado.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants