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

Jump to bottom

doCopyFileUsingNewIO does not follow symlinks #45

Closed
msimacek opened this issue Jul 11, 2018 · 3 comments
Closed

doCopyFileUsingNewIO does not follow symlinks #45

msimacek opened this issue Jul 11, 2018 · 3 comments
Assignees
@michael-o
Milestone
3.2.1

Comments

@msimacek
Copy link
Contributor

doCopyFileUsingNewIO does not follow symlinks which is inconsistent with doCopyFileUsingLegacyIO.
msimacek added a commit to msimacek/plexus-utils that referenced this issue Jul 11, 2018
@msimacek
Follow symlinks in NioFiles.copy
16d7825 
Make copying using NIO and legacy IO consistent with respect to
following symlinks in source.

Fixes: codehaus-plexus#45
@msimacek msimacek mentioned this issue Jul 11, 2018
Closed
@ebourg
Copy link

ebourg commented Nov 26, 2018

This causes the maven-dependency-plugin to fail in Debian, because relative symlink are copied instead of the actual targeted file. Once copied the symlink is invalid and points to nothing.

@michael-o michael-o self-assigned this Dec 4, 2018
@michael-o
Copy link
Member

Friends, ping me in a week or so if I haven't merge this yet. Can someone provide a link to the old solution? This looks straight forward to me.

@mizdebsk
Copy link
Contributor

This problem still affects plexus-utils 3.2.0.
@michael-o What do you mean by "the old solution"?

@michael-o michael-o added this to the 3.2.1 milestone Jul 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@michael-o michael-o

Labels
None yet
Projects
None yet
Milestone
3.2.1
Development

Successfully merging a pull request may close this issue.

Follow symlinks in NioFiles.copy msimacek/plexus-utils
4 participants
@ebourg @michael-o @mizdebsk @msimacek