On Fri, May 15, 2020 at 9:36 AM Petr Lautrbach <plautrba@xxxxxxxxxx> wrote: > > On Fri, May 15, 2020 at 02:41:11PM +0300, Topi Miettinen wrote: > > Environment variable TMPDIR may be already set for the user building > > and this could be equal to $XDG_RUNTIME_DIR or /tmp which are existing > > directories. Then when running 'make clean', there are unintended side > > effects: > > > > rm -rf /run/user/1000 > > rm: cannot remove '/run/user/1000/dconf/user': Permission denied > > rm: cannot remove '/run/user/1000/systemd': Permission denied > > rm: cannot remove '/run/user/1000/gnupg': Permission denied > > rm: cannot remove '/run/user/1000/dbus-1': Is a directory > > rm: cannot remove '/run/user/1000/inaccessible': Permission denied > > make[1]: *** [Makefile:68: clean] Error 1 > > > > Fix by always setting the variable. > > > > Signed-off-by: Topi Miettinen <toiwoton@xxxxxxxxx> > > Suggested-by: Petr Lautrbach <plautrba@xxxxxxxxxx> > > Acked-by: Petr Lautrbach <plautrba@xxxxxxxxxx> Should I merge this or do you have it?