[arch-projects] [devtools][PATCH 08/13] makechrootpkg: Add a comment warning about a bug in "sudo -i"

Luke Shumaker lukeshu at parabola.nu
Fri May 5 22:41:05 UTC 2017


The bug isn't currently triggered, but I accidentally did trigger when I
was trying to modify the command a bit.  I figure a "caution" sign would be
helpful to any future developers.
---
 makechrootpkg.in | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/makechrootpkg.in b/makechrootpkg.in
index d922fa0..2a19dbb 100644
--- a/makechrootpkg.in
+++ b/makechrootpkg.in
@@ -241,6 +241,10 @@ EOF
 # so no global variables
 _chrootbuild() {
 	. /etc/profile
+	# Beware, there are some stupid arbitrary rules on how you can
+	# use "$" in arguments to commands with "sudo -i".  ${foo} or
+	# ${1} is OK, but $foo or $1 isn't.
+	# https://bugzilla.sudo.ws/show_bug.cgi?id=765
 	sudo -iu builduser bash -c 'cd /startdir; makepkg "$@"' -bash "$@"
 }
 
-- 
2.12.2


More information about the arch-projects mailing list