Merge pull request #634 from darko-poljak/bugfix/consul_#633

bugfix: __consul fails if consul is already running #633

cdist/conf/type/__consul/gencode-remote

@@ -44,15 +44,17 @@ cksum_should=$(cat "$version_dir/cksum" | cut -d' ' -f1,2)
 cat << eof
     tmpdir=\$(mktemp -d --tmpdir="/tmp" "${__type##*/}.XXXXXXXXXX")
     curl -s -L "$source" > "\$tmpdir/$source_file_name"
-    unzip -p "\$tmpdir/$source_file_name" > "$destination"
+    unzip -p "\$tmpdir/$source_file_name" > "${destination}.tmp"
     rm -rf "\$tmpdir"
 
-    cksum_is=\$(cksum "$destination" | cut -d' ' -f1,2)
+    cksum_is=\$(cksum "${destination}.tmp" | cut -d' ' -f1,2)
     if [ "\$cksum_is" = "$cksum_should" ]; then
+        rm -f "${destination}"
+        mv "${destination}.tmp" "${destination}"
         chown root:root "$destination"
         chmod 755 "$destination"
     else
-        rm -f "$destination"
+        rm -f "${destination}.tmp"
         echo "Failed to verify checksum for $__object_name" >&2
         exit 1
     fi