From: Michael Prokop <mika@grml.org>
Date: Mon, 16 Apr 2007 10:56:10 +0000 (+0200)
Subject: Change logic of function execution.
X-Git-Tag: 0.7~35
X-Git-Url: https://git.grml.org/?p=grml-debootstrap.git;a=commitdiff_plain;h=a64dc69b9715e387d5d760c567eefd08c6f3cfce

Change logic of function execution.
---

diff --git a/debian/changelog b/debian/changelog
index 398f3c5..9fcc390 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,5 +1,6 @@
 grml-debootstrap (0.7) unstable; urgency=low
 
+  * Change logic of function execution.
   * Use aptitude instead of apt-get in chroot-script, but
     only if it's available.
   * Added additional check to grub code for running sed on
diff --git a/grml-debootstrap b/grml-debootstrap
index 6ba7842..8473abf 100644
--- a/grml-debootstrap
+++ b/grml-debootstrap
@@ -128,12 +128,12 @@ trap bailout 1 2 3 15
 
 # we want to execute all the functions only once, simple check for it:
 stage() {
-  if grep -q done "$STAGES/$1" 2>/dev/null ; then
-     ewarn "Notice: stage $1 has been executed already, skipping execution therefore." ; eend 0
-     return 1
-  else
+  if [ -n "$2" ] ; then
      echo "$2" > "$STAGES/$1"
      return 0
+  elif grep -q done "$STAGES/$1" 2>/dev/null ; then
+     ewarn "Notice: stage $1 has been executed already, skipping execution therefore." ; eend 0
+     return 1
   fi
 }
 # }}}
@@ -271,15 +271,12 @@ fscktool() {
 # }}}
 
 # now execute all the functions {{{
-  stage mkfs               && mkfs               && stage mkfs done               || bailout
-  stage tunefs             && tunefs             && stage tunefs done             || bailout
-  stage mount_target       && mount_target       && stage mount_target done       || bailout
-  stage debootstrap_system && debootstrap_system && stage debootstrap_system done || bailout
-  stage preparechroot      && preparechroot      && stage preparechroot done      || bailout
-  stage chrootscript       && chrootscript       && stage chrootscript done       || bailout
-  stage grub_install       && grub_install       && stage grub_install done       || bailout
-  stage umount_chroot      && umount_chroot      && stage umount_chroot done      || bailout
-  stage fscktool           && fscktool           && stage fscktool done           || bailout
+for i in mkfs tunefs mount_target debootstrap_system preparechroot \
+         chrootscript grub_install umount_chroot fscktool ; do
+    if stage $i ; then
+       $i && stage $i done || bailout
+    fi
+done
 # }}}
 
 # stages {{{