diff options
| author | Amir Alawi <83512437+amiralawi@users.noreply.github.com> | 2024-01-08 15:58:14 -0500 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-01-08 15:58:14 -0500 |
| commit | 4cbf74bd9b839501a4fd4bc5f39a25e216237b3c (patch) | |
| tree | 6cc25456391073e1820167cf896c2cf710dd18dc /lib/std/Build/Module.zig | |
| parent | ed410b9c1e9a09e674f862b5f3af9d207837a472 (diff) | |
| download | zig-4cbf74bd9b839501a4fd4bc5f39a25e216237b3c.tar.gz zig-4cbf74bd9b839501a4fd4bc5f39a25e216237b3c.zip | |
fix std.fs.Dir.makePath silent failure (#16878)
std.fs.dir.makePath silently failed if one of the items in the path already exists. For example:
cwd.makePath("foo/bar/baz")
Silently failing is OK if "bar" is already a directory - this is the intended use of makePath (like mkdir -p). But if bar is a file then the subdirectory baz cannot be created - the end result is that makePath doesn't do anything which should be a detectable error because baz is never created.
The existing code had a TODO comment that did not specifically cover this error, but the solution for this silent failure also accomplishes the TODO task - the code now stats "foo" and returns an appropriate error. The new code also handles potential race condition if "bar" is deleted/permissions changed/etc in between the initial makeDir and statFile calls.
Diffstat (limited to 'lib/std/Build/Module.zig')
0 files changed, 0 insertions, 0 deletions
