mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
From: "Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>
To: ptxdist@pengutronix.de
Subject: [ptxdist] [PATCH] treewide: Consistently use ':' instead to separate user and group for chown
Date: Fri, 16 Dec 2022 11:45:22 +0100	[thread overview]
Message-ID: <20221216104522.116336-1-u.kleine-koenig@pengutronix.de> (raw)

Since coreutils 9.1 chown warns when a '.' is used as separator. Fix this
consistently also for documentation and scripts that are expected to use
busybox's chown which (up to now) has no problem with '.'.

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
---
 config/busybox/Config.in               | 4 ++--
 config/busybox/networking/Config.in    | 2 +-
 projectroot/usr/sbin/mysqlinit         | 2 +-
 scripts/lib/ptxd_lib_dopermissions.awk | 4 ++--
 4 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/config/busybox/Config.in b/config/busybox/Config.in
index 199cc6ab272f..329e2f8e8a1b 100644
--- a/config/busybox/Config.in
+++ b/config/busybox/Config.in
@@ -267,10 +267,10 @@ config BUSYBOX_FEATURE_SUID_CONFIG
 
 	The file has to be owned by user root, group root and has to be
 	writeable only by root:
-		(chown 0.0 /etc/busybox.conf; chmod 600 /etc/busybox.conf)
+		(chown 0:0 /etc/busybox.conf; chmod 600 /etc/busybox.conf)
 	The busybox executable has to be owned by user root, group
 	root and has to be setuid root for this to work:
-		(chown 0.0 /bin/busybox; chmod 4755 /bin/busybox)
+		(chown 0:0 /bin/busybox; chmod 4755 /bin/busybox)
 
 	Robert 'sandman' Griebl has more information here:
 	<url: http://www.softforge.de/bb/suid.html >.
diff --git a/config/busybox/networking/Config.in b/config/busybox/networking/Config.in
index 3a3ad56d93f7..27e0f44ba811 100644
--- a/config/busybox/networking/Config.in
+++ b/config/busybox/networking/Config.in
@@ -959,7 +959,7 @@ config BUSYBOX_TELNETD
 	FEATURE_SUID enabled. And finally, you should make
 	certain that busybox has been installed setuid root:
 
-		chown root.root /bin/busybox
+		chown root:root /bin/busybox
 		chmod 4755 /bin/busybox
 
 	with all that done, telnetd _should_ work....
diff --git a/projectroot/usr/sbin/mysqlinit b/projectroot/usr/sbin/mysqlinit
index 72b6d1f10cf5..c98946025264 100644
--- a/projectroot/usr/sbin/mysqlinit
+++ b/projectroot/usr/sbin/mysqlinit
@@ -5,7 +5,7 @@ MYSQLD_OPTS=
 # since mySQL runs as a regular user, we need to give its working directory
 # to it.
 
-chown mysql.mysql "$1"
+chown mysql:mysql "$1"
 chmod 0755 "$1"
 
 # now let mySQL do its initializing job
diff --git a/scripts/lib/ptxd_lib_dopermissions.awk b/scripts/lib/ptxd_lib_dopermissions.awk
index 56e951b45b75..9ab45d4979dd 100755
--- a/scripts/lib/ptxd_lib_dopermissions.awk
+++ b/scripts/lib/ptxd_lib_dopermissions.awk
@@ -7,14 +7,14 @@ BEGIN {
 
 $1 ~ "[df]" {
 	printf("chmod %s	'.%s'\n" \
-	       "chown %s.%s	'.%s'\n", \
+	       "chown %s:%s	'.%s'\n", \
 	       $5, $2, $3, $4, $2);
 }
 
 $1 ~ "n" {
 	printf("if [ ! \\( -b '.%s' -o -c '.%s' -o -p '.%s' \\) ]; then " \
 	       "  mknod -m %s	'.%s'	%s %s %s\n" \
-	       "  chown %s.%s	'.%s'\n" \
+	       "  chown %s:%s	'.%s'\n" \
 	       "fi\n", \
 	       $2, $2, $2, $5, $2, $6, $7, $8, $3, $4, $2);
 }
-- 
2.38.1




             reply	other threads:[~2022-12-16 10:46 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-12-16 10:45 Uwe Kleine-König [this message]
2023-01-03 11:08 ` [ptxdist] [APPLIED] " Michael Olbrich

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20221216104522.116336-1-u.kleine-koenig@pengutronix.de \
    --to=u.kleine-koenig@pengutronix.de \
    --cc=ptxdist@pengutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox