concentrate on iteration in cdist-object-run-all; move all execution code into new executable bin/cdist-object-run

Signed-off-by: Steven Armstrong <steven@icarus.ethz.ch>

bin/cdist-object-run

@@ -0,0 +1,55 @@
+#!/bin/sh
+#
+# 2011 Nico Schottelius (nico-cdist at schottelius.org)
+# 2011 Steven Armstrong (steven-cdist at armstrong.cc)
+#
+# This file is part of cdist.
+#
+# cdist is free software: you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation, either version 3 of the License, or
+# (at your option) any later version.
+#
+# cdist is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with cdist. If not, see <http://www.gnu.org/licenses/>.
+#
+# 
+# For the given object:
+#  - run type explorers
+#  - generate code
+#  - copy object to target
+#  - execute code on target
+#
+
+. cdist-config
+[ $# -eq 2 ] || __cdist_usage "<target host> <object>"
+set -eu
+
+__cdist_target_host="$1"; shift
+__cdist_object="$1"; shift
+
+
+__cdist_type="$(__cdist_type_from_object "$__cdist_object")"
+
+# Check if type of object has >= 1 explorer
+__cdist_has_explorer="$(__cdist_type_has_explorer "$__cdist_type")"
+
+# Run the type explorers for the current object if any
+if [ "$__cdist_has_explorer" ]; then
+   cdist-object-explorer-run "$__cdist_target_host" "$__cdist_object"
+fi
+
+# Run the gencode scripts for the current object
+cdist-object-gencode-run "$__cdist_target_host" "$__cdist_object"
+
+# Transfer the current object to the target
+cdist-object-push "$__cdist_target_host" "$__cdist_object"
+
+# Run the code for the current object
+cdist-object-code-run "$__cdist_target_host" "$__cdist_object"
+

bin/cdist-object-run-all

@@ -19,11 +19,7 @@
 # along with cdist. If not, see <http://www.gnu.org/licenses/>.
 #
 # 
-# For each created object:
+# Run cdist-object-run for each created object.
-#  - run type explorers
-#  - generate code
-#  - copy object to target
-#  - execute code on target
 #
 
 . cdist-config
@@ -49,32 +45,19 @@ done < "$__cdist_objects"
 while [ $# -gt 0 ]; do
    __cdist_object="$1"; shift
 
-   __cdist_type="$(__cdist_type_from_object "$__cdist_object")"
-
    # Check if type of object has >= 1 explorer
    __cdist_has_explorer="$(__cdist_type_has_explorer "$__cdist_type")"
 
-   # If so, run explorers on remote side
+   # Transfer the type explorers if any
    if [ "$__cdist_has_explorer" ]; then
-      # Transfer the type explorers
       if ! grep -q "$__cdist_type" "$__cdist_types_pushed"; then
          cdist-type-explorer-push "$__cdist_target_host" "$__cdist_type"
          echo "$__cdist_type" >> "$__cdist_types_pushed" 
       fi 
-
-      # Run the type explorers for the current object
-      cdist-object-explorer-run "$__cdist_target_host" "$__cdist_object"
-
    fi
 
-   # Run the gencode scripts for the current object
+   # Process the given object
-   cdist-object-gencode-run "$__cdist_target_host" "$__cdist_object"
+   cdist-object-run "$__cdist_target_host" "$__cdist_object"
-
-   # Transfer the current object to the target
-   cdist-object-push "$__cdist_target_host" "$__cdist_object"
-
-   # Run the code for the current object
-   cdist-object-code-run "$__cdist_target_host" "$__cdist_object"
 
 done