]> git.sur5r.net Git - i3/i3/commit
mkdirp: do not throw an error if directory exists
authorTheo Buehler <theo@math.ethz.ch>
Sat, 23 May 2015 11:12:18 +0000 (13:12 +0200)
committerMichael Stapelberg <michael@stapelberg.de>
Thu, 30 Jul 2015 19:41:24 +0000 (21:41 +0200)
commitc0f685e7bc834e8ffd266c79d04451b995804b14
treec0f744a4c7808a26a4c1257a58e229286b44b78f
parentfbcb227537a15d1ef5873500203ae7394a3b3d8f
mkdirp: do not throw an error if directory exists

If I restart i3 4.10.2 twice, e.g. with

$ i3-msg restart; sleep 3; i3-msg restart

the second time I get the following two errors:

05/22/15 10:46:03 - ERROR: mkdir(/tmp/i3-theo.toAK7N) failed: File exists
05/22/15 10:46:03 - ERROR: Could not create "/tmp/i3-theo.toAK7N" for storing the restart layout, layout will be lost.

The first one is from mkdirp() in src/ipc.c and the second one is from
store_restart_layout() in src/util.c.

Notice that I do _not_ get the ``open()'' or ``Could not write restart layout to
...'' error messages, so the layout writing code after line 260 in
store_restart_layout() succeeded and the layout isn't actually lost.  Thus,
these error messages are a bit misleading, especially the second one (which is
triggered by the failure of mkdirp()).

POSIX says about `mkdir -p':

``Each dir operand that names an existing directory shall be ignored without
error.''

Therefore, I suggest the following simple patch that makes mkdirp() succeed if
the named file exists and actually is a directory.  This silences the second
error as well.
libi3/mkdirp.c