| View Issue Details [ Jump to Notes ] | [ Print ] | ||||||||
| ID | Project | Category | View Status | Date Submitted | Last Update | ||||
| 0012433 | CMake | CMake | public | 2011-08-30 14:44 | 2013-01-09 10:57 | ||||
| Reporter | Erik Greenwald | ||||||||
| Assigned To | David Cole | ||||||||
| Priority | normal | Severity | minor | Reproducibility | always | ||||
| Status | closed | Resolution | no change required | ||||||
| Platform | OS | FreeBSD/MacOSX/Linux | OS Version | ||||||
| Product Version | CMake 2.8.5 | ||||||||
| Target Version | Fixed in Version | CMake 2.8.10 | |||||||
| Summary | 0012433: -E rename across FS's causes cross-device link failure | ||||||||
| Description | rename() throws EXDEV when done between devices (different partitions, nfs to local, etc). Patch provided to do simple copy when EXDEV is caught. | ||||||||
| Tags | No tags attached. | ||||||||
| Attached Files | |||||||||
| Relationships | |
| Relationships |
| Notes | |
|
(0027312) David Cole (manager) 2011-08-31 12:27 |
Please see my response on the CMake mailing list: http://www.cmake.org/pipermail/cmake/2011-August/046054.html [^] I don't think a simple copy is appropriate for all cases, so I don't think we should accept this patch into CMake. |
|
(0027313) David Cole (manager) 2011-08-31 12:29 |
Additionally, "cmake -E rename" is supposed to be guaranteed to be "atomic" -- adding this code makes it "mostly atomic" and would negate the intent of some significant callers of rename... |
|
(0030270) David Cole (manager) 2012-08-11 11:31 |
See previous notes. Caller should do a copy after a failed rename if that's what the caller wants. It's not CMake's business to automatically do a copy if a rename operation fails. |
|
(0032044) Robert Maynard (manager) 2013-01-09 10:57 |
Closing resolved issues that have not been updated in more than 4 months. |
| Notes |
| Issue History | |||
| Date Modified | Username | Field | Change |
| 2011-08-30 14:44 | Erik Greenwald | New Issue | |
| 2011-08-30 14:44 | Erik Greenwald | File Added: 0001-Cope-with-cross-device-links-add-simple-copy.patch | |
| 2011-08-31 12:27 | David Cole | Note Added: 0027312 | |
| 2011-08-31 12:29 | David Cole | Note Added: 0027313 | |
| 2012-08-11 11:30 | David Cole | Assigned To | => David Cole |
| 2012-08-11 11:30 | David Cole | Status | new => assigned |
| 2012-08-11 11:31 | David Cole | Note Added: 0030270 | |
| 2012-08-11 11:31 | David Cole | Status | assigned => resolved |
| 2012-08-11 11:31 | David Cole | Fixed in Version | => CMake 2.8.10 |
| 2012-08-11 11:31 | David Cole | Resolution | open => no change required |
| 2013-01-09 10:57 | Robert Maynard | Note Added: 0032044 | |
| 2013-01-09 10:57 | Robert Maynard | Status | resolved => closed |
| Issue History |
| Copyright © 2000 - 2018 MantisBT Team |