roken: do not include the rk_rename() implementation on unix 91/12091/2
authorMichael Meffie <mmeffie@sinenomine.net>
Thu, 5 Nov 2015 21:29:05 +0000 (16:29 -0500)
committerBenjamin Kaduk <kaduk@mit.edu>
Wed, 27 Apr 2016 02:40:14 +0000 (22:40 -0400)
commit8bf89fdbc251822b6a3149088f5634bb40e5c225
tree6ef3b2f1e2c628dfa5e2d38edbcaea0ff27ca6e2
parent40dfd90a9f29ea56a871449172f809c4ae3cd4f6
roken: do not include the rk_rename() implementation on unix

libroken provides roken/rename.c for platforms where the native rename()
implementation does not replace the target if it already exists.  As designed,
rk_rename() should be used instead of rename() everywhere and rk_rename()
is #defined to be rename() on platforms where this fix is not necessary.

Do not include the rk_rename() implementation on platforms which do not need
the rk_rename since it is not used on those platforms.

Note: This fix also avoids a recursive rename(). As currently implemented, the
rk_rename() function is redefined to rename() within the roken/rename.c module
when RENAME_DOES_NOT_UNLINK is not defined. This can mask the standard library
rename() and leads to a recursive call to rename().

Change-Id: I47a1fcd21939b161aaa7df7ffab26dc84e7b75ed
Reviewed-on: https://gerrit.openafs.org/12091
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Mark Vitale <mvitale@sinenomine.net>
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
src/roken/Makefile.in