Author: adam-guest
Date: 2008-06-26 21:39:54 +0000 (Thu, 26 Jun 2008)
New Revision: 1533

Modified:
   trunk/debian/changelog
   trunk/scripts/checkbashisms.pl
Log:
Improve the suggested replacement for $UID and $EUID (Closes: #488190)

Modified: trunk/debian/changelog
===================================================================
--- trunk/debian/changelog      2008-06-26 18:16:27 UTC (rev 1532)
+++ trunk/debian/changelog      2008-06-26 21:39:54 UTC (rev 1533)
@@ -13,6 +13,7 @@
     + Add a --posix option which flags constructs not supported by POSIX but
       permitted by Debian Policy (echo -n, local (with no options or
       assignments) and test / [ using -a or -o )
+    + Improve the suggested replacement for $UID and $EUID (Closes: #488190)
   * Debbugs.pm: Move initialisation of the SOAP object in to a common routine
     and set the underlying transport object to honour proxies from the
     environment (Closes: #487592)

Modified: trunk/scripts/checkbashisms.pl
===================================================================
--- trunk/scripts/checkbashisms.pl      2008-06-26 18:16:27 UTC (rev 1532)
+++ trunk/scripts/checkbashisms.pl      2008-06-26 21:39:54 UTC (rev 1533)
@@ -478,8 +478,8 @@
        qr'\$\{?(OS|MACH)TYPE\}?\b'   => q<$(OS|MACH)TYPE>,
        qr'\$\{?HOST(TYPE|NAME)\}?\b' => q<$HOST(TYPE|NAME)>,
        qr'\$\{?DIRSTACK\}?\b'        => q<$DIRSTACK>,
-       qr'\$\{?EUID\}?\b'          => q<$EUID should be "id -u">,
-       qr'\$\{?UID\}?\b'                   => q<$UID should be "id -ru">,
+       qr'\$\{?EUID\}?\b'            => q<$EUID should be "$(id -u)">,
+       qr'\$\{?UID\}?\b'              => q<$UID should be "$(id -ru)">,
        qr'\$\{?SECONDS\}?\b'       => q<$SECONDS>,
        qr'\$\{?BASH_[A-Z]+\}?\b'     => q<$BASH_SOMETHING>,
        qr'\$\{?SHELLOPTS\}?\b'       => q<$SHELLOPTS>,



-- 
To unsubscribe, send mail to [EMAIL PROTECTED]

Reply via email to