Try to determine whether docker container worked.

pjcj · May 28, 2014 · 9064cda · 9064cda
1 parent 88c587a
commit 9064cda
Showing 1 changed file with 17 additions and 15 deletions.
diff --git a/utils/dc b/utils/dc
@@ -130,21 +130,23 @@ case "$1" in
                     pjcj/cpancover dc cpancover-build-module $module)
         # https://github.com/dotcloud/docker/issues/3986
         docker.io wait "$name"
-        docker.io logs "$name"
-        local_staging="$staging/$name"
-        sudo docker.io cp "$name:/staging" "$local_staging"
-        if [ -d "$local_staging" ]; then
-            sudo chmod -R 755 "$local_staging"
-            sudo find "$local_staging" -type f -exec chmod 644 {} \;
-            sudo chown -R pjcj:pjcj "$local_staging"
-            cd "$local_staging"/*
-            for f in *; do
-                if [ -d $f ]; then
-                    rm -rf "$staging"/$f
-                    mv $f "$staging"
-                fi
-            done
-            rm -r "$local_staging"
+        if [ $? = 0 ]; then
+            docker.io logs "$name"
+            local_staging="$staging/$name"
+            sudo docker.io cp "$name:/staging" "$local_staging"
+            if [ -d "$local_staging" ]; then
+                sudo chmod -R 755 "$local_staging"
+                sudo find "$local_staging" -type f -exec chmod 644 {} \;
+                sudo chown -R pjcj:pjcj "$local_staging"
+                cd "$local_staging"/*
+                for f in *; do
+                    if [ -d $f ]; then
+                        rm -rf "$staging"/$f
+                        mv $f "$staging"
+                    fi
+                done
+                rm -r "$local_staging"
+            fi
         fi
         docker.io rm "$name"
         ;;