Skip to content

Commit

Permalink
docker: add missing negation of the exit status
Browse files Browse the repository at this point in the history
Broken with commit cc988ce
  • Loading branch information
mlschroe committed May 31, 2021
1 parent 088bdfc commit 12b4fc8
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion build-recipe-docker
Original file line number Diff line number Diff line change
Expand Up @@ -60,6 +60,7 @@ recipe_build_docker() {
base_image_path=$(find containers -regextype egrep -regex ".*\.(tgz|tar|tar\.xz|tar\.gz)$" -print -quit)
test -f "$base_image_path" || cleanup_and_exit 1 "base image not found"

mkdir -p "$BUILD_ROOT/$TOPDIR/SOURCES/repos"
if test "$BUILDENGINE" = podman; then
DOCKER_TOOL=podman
DOCKER_CMD="$BUILD_DIR/call-podman --root $BUILD_ROOT"
Expand Down Expand Up @@ -178,7 +179,7 @@ recipe_build_docker() {
FILENAME="$FILENAME.${BUILD_ARCH%%:*}"
test -n "$RELEASE" && FILENAME="$FILENAME-$RELEASE"
echo "Saving image $FIRSTTAG to $FILENAME.tar"
if $DOCKER_CMD save --output "$TOPDIR/DOCKER/$FILENAME.tar" "$FIRSTTAG" ; then
if ! $DOCKER_CMD save --output "$TOPDIR/DOCKER/$FILENAME.tar" "$FIRSTTAG" ; then
cleanup_and_exit 1 "$DOCKER_TOOL save command failed"
fi

Expand Down

0 comments on commit 12b4fc8

Please sign in to comment.