[ARVADOS] updated: 2b9d07af18b58b8a9c73620d58c07a10703802ea

Git user git at public.curoverse.com
Mon Aug 22 15:15:12 EDT 2016


Summary of changes:
 build/go-package-scripts/postinst | 12 ++++++------
 build/go-package-scripts/prerm    |  6 +++---
 2 files changed, 9 insertions(+), 9 deletions(-)

       via  2b9d07af18b58b8a9c73620d58c07a10703802ea (commit)
      from  cfabe4198e0bf332c102b45a16a59c12f99cba4d (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 2b9d07af18b58b8a9c73620d58c07a10703802ea
Author: Tom Clegg <tom at curoverse.com>
Date:   Mon Aug 22 15:14:49 2016 -0400

    9706: Use ${} interpolation syntax consistently.

diff --git a/build/go-package-scripts/postinst b/build/go-package-scripts/postinst
index 8559a97..0663465 100755
--- a/build/go-package-scripts/postinst
+++ b/build/go-package-scripts/postinst
@@ -5,25 +5,25 @@ set -e
 # NOTE: This package name detection will only work on Debian.
 # If this postinst script ever starts doing work on Red Hat,
 # we'll need to adapt this code accordingly.
-script="$(basename "$0")"
+script="$(basename "${0}")"
 pkg="${script%.postinst}"
 systemd_unit="${pkg}.service"
 
 case "${1}" in
     configure)
         if [ -e /run/systemd/system ]; then
-            eval "$(systemctl -p UnitFileState show "$systemd_unit")"
-            case "$UnitFileState" in
+            eval "$(systemctl -p UnitFileState show "${systemd_unit}")"
+            case "${UnitFileState}" in
                 disabled)
                     # Failing to enable or start the service is not a
                     # package error, so don't let errors here
                     # propagate up.
-                    systemctl enable "$systemd_unit" || true
-                    systemctl start "$systemd_unit" || true
+                    systemctl enable "${systemd_unit}" || true
+                    systemctl start "${systemd_unit}" || true
                     ;;
                 enabled)
                     systemctl daemon-reload || true
-                    systemctl reload-or-try-restart "$systemd_unit" || true
+                    systemctl reload-or-try-restart "${systemd_unit}" || true
                     ;;
             esac
         fi
diff --git a/build/go-package-scripts/prerm b/build/go-package-scripts/prerm
index 4e1399d..02772e4 100755
--- a/build/go-package-scripts/prerm
+++ b/build/go-package-scripts/prerm
@@ -5,15 +5,15 @@ set -e
 # NOTE: This package name detection will only work on Debian.
 # If this prerm script ever starts doing work on Red Hat,
 # we'll need to adapt this code accordingly.
-script="$(basename "$0")"
+script="$(basename "${0}")"
 pkg="${script%.prerm}"
 systemd_unit="${pkg}.service"
 
 case "${1}" in
     remove)
         if [ -e /run/systemd/system ]; then
-            systemctl stop "$systemd_unit" || true
-            systemctl disable "$systemd_unit" || true
+            systemctl stop "${systemd_unit}" || true
+            systemctl disable "${systemd_unit}" || true
         fi
         ;;
 esac

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list