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 | 0001-Cope-with-cross-device-links-add-simple-copy.patch [^] (1,425 bytes) 2011-08-30 14:44 [Show Content] | ||||||||
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 |