Merge lp:~jamesodhunt/ubiquity/bug-944191 into lp:ubiquity
Proposed by
James Hunt
Status: | Superseded | ||||
---|---|---|---|---|---|
Proposed branch: | lp:~jamesodhunt/ubiquity/bug-944191 | ||||
Merge into: | lp:ubiquity | ||||
Diff against target: |
52 lines (+20/-2) (has conflicts) 2 files modified
debian/changelog (+10/-1) scripts/install.py (+10/-1) Text conflict in debian/changelog |
||||
To merge this branch: | bzr merge lp:~jamesodhunt/ubiquity/bug-944191 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Colin Watson | Pending | ||
Review via email: mp+98489@code.launchpad.net |
This proposal supersedes a proposal from 2012-03-20.
This proposal has been superseded by a proposal from 2012-03-20.
Description of the change
* scripts/install.py: Ignore mkdir failure if directory does (now)
exist to accommodate update-apt-cache running in parallel with
copy_all() (LP: #944191).
(Updated with Colins feedback).
To post a comment you must log in.
The style in ubiquity is to check specifically for OSError, rather than for the more general EnvironmentError. Doing so is safe in this case; you aren't going to get anything else from os.mkdir.
I don't think you need to recheck os.path.isdir; simply having got EEXIST from os.mkdir implies that. I would just write this as 'if e.errno != errno.EEXIST'.