Merge lp://staging/~mthaddon/codetree/overwrite-option-symlinks into lp://staging/~codetree-maintainers/codetree/master
Proposed by
Tom Haddon
Status: | Merged |
---|---|
Approved by: | Matthew Wedgwood |
Approved revision: | 35 |
Merge reported by: | Matthew Wedgwood |
Merged at revision: | not available |
Proposed branch: | lp://staging/~mthaddon/codetree/overwrite-option-symlinks |
Merge into: | lp://staging/~codetree-maintainers/codetree/master |
Diff against target: |
40 lines (+12/-3) 2 files modified
codetree/fileutils.py (+10/-2) codetree/handlers.py (+2/-1) |
To merge this branch: | bzr merge lp://staging/~mthaddon/codetree/overwrite-option-symlinks |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Matthew Wedgwood | Approve | ||
David Ames (community) | Approve | ||
Review via email: mp+194519@code.staging.launchpad.net |
Description of the change
Allow us to use the overwrite option for a symlink. This will still fail if the destination already exists as another file, but if it's a symlink it will overwrite it.
The specific use case for this is in mojo if you're running the collect phase against a pre-existing workspace. Without being able to overwrite any pre-existing symlinks this will fail. Happy to discuss other options if appropriate.
To post a comment you must log in.
+1 from me