Merge lp:~nha/widelands/filesystem into lp:widelands
Proposed by
Nicolai Hähnle
Status: | Merged | ||||
---|---|---|---|---|---|
Merged at revision: | 5358 | ||||
Proposed branch: | lp:~nha/widelands/filesystem | ||||
Merge into: | lp:widelands | ||||
Diff against target: |
166 lines (+45/-32) 4 files modified
src/io/filesystem/disk_filesystem.cc (+23/-21) src/io/filesystem/filesystem.h (+2/-2) src/io/filesystem/layered_filesystem.cc (+14/-3) src/wlapplication.cc (+6/-6) |
||||
To merge this branch: | bzr merge lp:~nha/widelands/filesystem | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Tino | Approve | ||
Widelands Developers | crossplatform | Pending | |
Review via email: mp+25802@code.launchpad.net |
Description of the change
Clean up the recursiveness of EnsureDirectory
I'm a bit surprised nobody else has run into the non-creation of ~/.widelands issue, so maybe it is done somewhere I have missed, but in a way that makes it fail given the changes to EnsureDirectory
To post a comment you must log in.
test on win32:
- code compiles
- wl finds existing .widelands dir in %Userhome%
- wl creates .widelands if it does not exist