Fix: chown: Use double colon for OWNER:GROUP
PATCH 1/1 Fix: chown: Use double colon for OWNER:GROUP
Rationale - excerp from info chown
====================================
OWNER‘:’GROUP
If the OWNER is followed by a colon and a GROUP (a group name or numeric group ID), with no spaces between them, the group ownership of the files is changed as well (to GROUP).
Some older scripts may still use ‘.’ in place of the ‘:’ separator.
POSIX 1003.1-2001 (*note Standards conformance::) does not require support for that, but for backward compatibility GNU ‘chown’ supports ‘.’ so long as no ambiguity results. New scripts should avoid the use of ‘.’ because it is not portable, and because it has undesirable results if the entire OWNER‘.’GROUP happens to identify a user whose name contains ‘.’.
Signed-off-by Kosta Zertsekel <[email protected]>
--- meta/recipes-core/images/build-appliance-image_15.0.0.bb | 6 +++--- .../initscripts/initscripts-1.0/populate-volatile.sh | 4 ++-- meta/recipes-core/sysvinit/sysvinit_2.88dsf.bb | 2 +- 3 files changed, 6 insertions(+), 6 deletions(-)
meta/recipes-core/images/build-appliance-image_15.0.0.bb +3 -3
@@ -79,7 +79,7 @@ fakeroot do_populate_poky_src () { echo "# export ALL_PROXY=https://proxy.example.com:8080" >> ${IMAGE_ROOTFS}/home/builder/.bashrc echo "# export ALL_PROXY=socks://socks.example.com:1080" >> ${IMAGE_ROOTFS}/home/builder/.bashrc - chown -R builder.builder ${IMAGE_ROOTFS}/home/builder/poky + chown -R builder:builder ${IMAGE_ROOTFS}/home/builder/poky chmod -R ug+rw ${IMAGE_ROOTFS}/home/builder/poky # Assume we will need CDROM to install guest additions
@@ -107,8 +107,8 @@ fakeroot do_populate_poky_src () { pip3_install_params="${pip3_install_params} --proxy ${http_proxy}" fi pip3 install ${pip3_install_params} - chown -R builder.builder ${IMAGE_ROOTFS}/home/builder/.local - chown -R builder.builder ${IMAGE_ROOTFS}/home/builder/.cache + chown -R builder:builder ${IMAGE_ROOTFS}/home/builder/.local + chown -R builder:builder ${IMAGE_ROOTFS}/home/builder/.cache } IMAGE_PREPROCESS_COMMAND += "do_populate_poky_src; "
meta/recipes-core/initscripts/initscripts-1.0/populate-volatile.sh +2 -2
@@ -37,7 +37,7 @@ create_file() { } EXEC=" ${EXEC} - chown ${TUSER}.${TGROUP} $1 || echo \"Failed to set owner -${TUSER}- for -$1-.\" >/dev/tty0 2>&1; + chown ${TUSER}:${TGROUP} $1 || echo \"Failed to set owner -${TUSER}- for -$1-.\" >/dev/tty0 2>&1; chmod ${TMODE} $1 || echo \"Failed to set mode -${TMODE}- for -$1-.\" >/dev/tty0 2>&1 " test "$VOLATILE_ENABLE_CACHE" = yes && echo "$EXEC" >> /etc/volatile.cache.build
@@ -60,7 +60,7 @@ create_file() { mk_dir() { EXEC=" mkdir -p \"$1\"; - chown ${TUSER}.${TGROUP} $1 || echo \"Failed to set owner -${TUSER}- for -$1-.\" >/dev/tty0 2>&1; + chown ${TUSER}:${TGROUP} $1 || echo \"Failed to set owner -${TUSER}- for -$1-.\" >/dev/tty0 2>&1; chmod ${TMODE} $1 || echo \"Failed to set mode -${TMODE}- for -$1-.\" >/dev/tty0 2>&1 " test "$VOLATILE_ENABLE_CACHE" = yes && echo "$EXEC" >> /etc/volatile.cache.build
meta/recipes-core/sysvinit/sysvinit_2.88dsf.bb +1 -1
@@ -105,6 +105,6 @@ do_install () { install -d ${D}${sysconfdir}/default/volatiles install -m 0644 ${WORKDIR}/01_bootlogd ${D}${sysconfdir}/default/volatiles - chown root.shutdown ${D}${base_sbindir}/halt ${D}${base_sbindir}/shutdown + chown root:shutdown ${D}${base_sbindir}/halt ${D}${base_sbindir}/shutdown chmod o-x,u+s ${D}${base_sbindir}/halt ${D}${base_sbindir}/shutdown }
Signature · 1 line
2.17.1