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@.