Changeset 537 for trunk

Timestamp:

08/21/2011 18:28:01 (9 months ago)

Author:

blee

Message:

Consolidate confsync revision comparison code.

This also catches the case where the revision numbers are equal
but the last action was an install, in which case confsync should run.

Fixes #147

File:

• trunk/confsync.in (modified) (2 diffs)

trunk/confsync.in

@@ -1 @@
-#!@BASH@
+#!@BASH@ -x
 #
 # Copyright (c) 2008, Christopher Cowart and contributors
@@ -117 +117 @@
 last_action=$(conf_lastact)
 
-if [ "x$last_action" = "xinstall" ] && [ $tar_revision -lt $sys_revision ]
-then
-    echo "Error: File system may be dirty." \
-        "A commit operation is recommended." >&2
-    echo "Error: $tarpath is out of date. Exiting without making changes." >&2
-    conf_markdirty
-    conf_cleanExit 1
-fi
-
-if [ -n "$sys_revision" ] ; then
-    if $FORCE && [ $tar_revision -lt $sys_revision ] ; then
-        echo "Error: $tarpath is out of date. Exiting without making changes." >&2
+if [ -n "${sys_revision}" ] ; then
+    if [ "${tar_revision}" -lt "${sys_revision}" ]; then
+        echo "Error: ${tarpath} is out of date. Exiting without making changes." >&2
+        conf_markdirty
         conf_cleanExit 1
-    elif ! $FORCE && [ $tar_revision -le $sys_revision ] ; then
-        $QUIET || echo "System is up to date." | conf_logger
-        conf_cleanExit 0
+    elif ! ${FORCE} && [ "${tar_revision}" -eq "${sys_revision}" ]; then
+        if [ "${last_action}" = "commit" ]; then
+            ${QUIET} || echo "System is up to date." | conf_logger
+            conf_cleanExit 0
+        fi
     fi
 fi