CVSROOT:        /cvs
Module name:    src
Changes by:     schwa...@cvs.openbsd.org        2019/02/08 05:53:44

Modified files:
        usr.bin/xinstall: install.1 xinstall.c 

Log message:
Fix a race condition: do not unlink(2) a file and then open(2) it
with O_CREAT|O_EXCL; instead, always create it with a temporary name,
then rename(2) it into place atomically.  For example, the race caused
failures in parallel builds that (foolishly) install the same file twice.

This patch makes the -S option a no-op,
making install(1) always behave like -S used to.

Based on a minimally different patch
from Lauri Tirkkonen <lotheac at iki dot fi>,
and including a manual page tweak from deraadt@.

OK deraadt@; "seems the right thing to do" tedu@.

Reply via email to