Skip to content

Don't dereference symlinks when copying files#20

Closed
brechtm wants to merge 1 commit intosphinx-contrib:masterfrom
brechtm:copy_symlinks
Closed

Don't dereference symlinks when copying files#20
brechtm wants to merge 1 commit intosphinx-contrib:masterfrom
brechtm:copy_symlinks

Conversation

@brechtm
Copy link
Contributor

@brechtm brechtm commented Aug 19, 2016

This prevents FileNotFoundError exceptions on broken symlinks.
Symlinks in a repository should be relative anyway, so there is no
advantage to dereferencing them.

This prevents FileNotFoundError exceptions on broken symlinks.
Symlinks in a repository should be relative anyway, so there is no
advantage to dereferencing them.
@coveralls
Copy link

coveralls commented Aug 19, 2016

Coverage Status

Coverage decreased (-0.1%) to 99.753% when pulling 91083f8 on brechtm:copy_symlinks into 5209695 on Robpol86:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.1%) to 99.753% when pulling 91083f8 on brechtm:copy_symlinks into 5209695 on Robpol86:master.

@Robpol86
Copy link
Collaborator

That keyword arg doesn't exist for .copy() in Python 2.7.

Robpol86 added a commit that referenced this pull request Aug 22, 2016
Skipping broken symlinks, but not all symlinks.

Reported in #20
@Robpol86
Copy link
Collaborator

Fixed in 8dbff4e. Thanks for reporting this.

@Robpol86 Robpol86 closed this Aug 22, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants