From 45c8f13d6559ef546bbf2d1390ad0b1145191da4 Mon Sep 17 00:00:00 2001 From: Benjamin Drung Date: Sat, 30 Oct 2010 20:36:42 +0200 Subject: [PATCH] syncpackage: Print an error message if the source-only build fails (LP: #668749). --- debian/changelog | 3 ++- syncpackage | 7 ++++++- 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/debian/changelog b/debian/changelog index 7ca032d..444af9e 100644 --- a/debian/changelog +++ b/debian/changelog @@ -35,8 +35,9 @@ ubuntu-dev-tools (0.105) UNRELEASED; urgency=low - Don't add quotation marks to changelog entries (LP: #668805). - Show a error message instead of raising an error if Ubuntu contains a newer version. + - Print an error message if the source-only build fails (LP: #668749). - -- Benjamin Drung Sat, 30 Oct 2010 20:10:39 +0200 + -- Benjamin Drung Sat, 30 Oct 2010 20:34:55 +0200 ubuntu-dev-tools (0.104) experimental; urgency=low diff --git a/syncpackage b/syncpackage index 9dcd36c..164cca2 100755 --- a/syncpackage +++ b/syncpackage @@ -372,7 +372,12 @@ def sync_dsc(script_name, dscurl, debian_dist, release, name, email, bugs, cmd += ["-k" + keyid] if verbose: print_command(script_name, cmd) - subprocess.check_call(cmd, env=env) + returncode = subprocess.call(cmd, env=env) + if returncode != 0: + print >> sys.stderr, ("%s: Error: Source-only build with debuild " + "failed. Please check build log above.") % \ + (script_name) + sys.exit(1) def get_debian_dscurl(package, dist, release, version=None, component=None): if dist is None: