summaryrefslogtreecommitdiffhomepage
path: root/docs
diff options
context:
space:
mode:
authorAlex Colomar <a.colomar@f5.com>2022-10-11 16:00:06 +0200
committerAlejandro Colomar <alx@nginx.com>2022-10-20 15:50:05 +0200
commitf93361979a9f612b59470640c3566f5cb66c3eaf (patch)
tree25a44efa6f0f0303584ffb2d48ecc4339af88d8b /docs
parentbbf1f4da0fe19c51253400a2cef58b8bac28fb25 (diff)
downloadunit-f93361979a9f612b59470640c3566f5cb66c3eaf.tar.gz
unit-f93361979a9f612b59470640c3566f5cb66c3eaf.tar.bz2
Avoided modifying existing directories at 'make install'.
'install -d' has an issue compared to 'mkdir -p': it doesn't respect existing directories. It will set the ownership, file mode, and SELinux contexts (and any other property that would be set by install(1) to a newly-created directory), overwriting any existing properties of the existing directory. 'mkdir -p' doesn't have this issue: it is a no-op if the directory exists. However, it's not an ideal solution either, since it can't be used to set the properties (owner, mode, ...) of a newly-created directory. Therefore, the best solution is to use install(1), but only after making sure that the directory doesn't exist with test(1). Reported-by: Andrew Clayton <a.clayton@nginx.com> Reported-by: Alejandro Colomar <alx@nginx.com> Closes: <https://github.com/nginx/unit/issues/769> Signed-off-by: Alejandro Colomar <alx@nginx.com> Tested-by: Andrew Clayton <a.clayton@nginx.com> Reviewed-by: Andrew Clayton <a.clayton@nginx.com>
Diffstat (limited to '')
0 files changed, 0 insertions, 0 deletions