diff options
| author | Chandan Singh <csingh43@bloomberg.net> | 2018-10-05 21:12:44 -0400 |
|---|---|---|
| committer | Chandan Singh <csingh43@bloomberg.net> | 2018-10-05 21:14:24 -0400 |
| commit | 2d09c6c1591e5635db7444d1c99d4e85ef4a9ead (patch) | |
| tree | 3f87c312d0c07f1c331b3ddf571a7197a8d9b488 | |
| parent | d3921ab8bd515fb3dc890c831b7fb33d34d6de05 (diff) | |
| download | buildstream-chandan/fix-bst-docker-import-logging.tar.gz | |
bst-docker-import: Consistently use stderr for all logschandan/fix-bst-docker-import-logging
In !857, we added `contrib/bst-docker-import` script but it has a small
issue that some of logs go to stdout while others go to stderr. Fix it
so that all logging is done on stderr.
| -rwxr-xr-x | contrib/bst-docker-import | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/contrib/bst-docker-import b/contrib/bst-docker-import index a451b8770..dfb16d7ce 100755 --- a/contrib/bst-docker-import +++ b/contrib/bst-docker-import @@ -93,10 +93,10 @@ echo "INFO: Checking out $element ..." >&2 $bst_cmd checkout --tar "$element" "$checkout_tar" || die "Failed to checkout $element" echo "INFO: Successfully checked out $element" >&2 -echo "INFO: Importing Docker image ..." +echo "INFO: Importing Docker image ..." >&2 "${docker_import_cmd[@]}" "$checkout_tar" "$docker_image_tag" || die "Failed to import Docker image from tarball" -echo "INFO: Successfully import Docker image $docker_image_tag" +echo "INFO: Successfully import Docker image $docker_image_tag" >&2 -echo "INFO: Cleaning up ..." +echo "INFO: Cleaning up ..." >&2 rm "$checkout_tar" || die "Failed to remove $checkout_tar" -echo "INFO: Clean up finished" +echo "INFO: Clean up finished" >&2 |