[Notes] [Git][BuildStream/buildstream][master] 2 commits: bst-docker-import: Consistently use stderr for all logs

[Chandan Singh <gitlab mg gitlab com>]

Title: GitLab

Chandan Singh pushed to branch master at BuildStream / buildstream

Commits:

• 3ca487b8
  by Chandan Singh at 2018-10-08T18:25:39Z

  bst-docker-import: Consistently use stderr for all logs
  
  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.
  

• 49df3d75
  by Chandan Singh at 2018-10-08T18:58:54Z

  Merge branch 'chandan/fix-bst-docker-import-logging' into 'master'
  
  bst-docker-import: Consistently use stderr for all logs
  
  See merge request BuildStream/buildstream!863

1 changed file:

• contrib/bst-docker-import

Changes:

contrib/bst-docker-import

---

...	...	@@ -93,10 +93,10 @@ echo "INFO: Checking out $element ..." >&2
93	93	$bst_cmd checkout --tar "$element" "$checkout_tar" || die "Failed to checkout $element"
94	94	echo "INFO: Successfully checked out $element" >&2
95	95
96		-echo "INFO: Importing Docker image ..."
	96	+echo "INFO: Importing Docker image ..." >&2
97	97	"${docker_import_cmd[@]}" "$checkout_tar" "$docker_image_tag" || die "Failed to import Docker image from tarball"
98		-echo "INFO: Successfully import Docker image $docker_image_tag"
	98	+echo "INFO: Successfully import Docker image $docker_image_tag" >&2
99	99
100		-echo "INFO: Cleaning up ..."
	100	+echo "INFO: Cleaning up ..." >&2
101	101	rm "$checkout_tar" || die "Failed to remove $checkout_tar"
102		-echo "INFO: Clean up finished"
	102	+echo "INFO: Clean up finished" >&2

—
View it on GitLab.
You're receiving this email because of your account on gitlab.com. If you'd like to receive fewer emails, you can adjust your notification settings.