laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-ci/+/38699?usp=email )

Change subject: gerrit_binpkgs: display hint for OBS build status
......................................................................

gerrit_binpkgs: display hint for OBS build status

Binary packages may fail to build if dependency binary packages are not
build yet. Display the link where the status of those packages can be
seen.

Suggested-by: Pau Espin Pedrol <[email protected]>
Change-Id: I6a2d7c27f6cd8f0e688aefacc819d3760f21e02e
---
M scripts/obs/gerrit_binpkgs.sh
1 file changed, 12 insertions(+), 3 deletions(-)

Approvals:
  laforge: Looks good to me, approved
  fixeria: Looks good to me, but someone else must approve
  Jenkins Builder: Verified




diff --git a/scripts/obs/gerrit_binpkgs.sh b/scripts/obs/gerrit_binpkgs.sh
index 4e3b0f3..443c33f 100755
--- a/scripts/obs/gerrit_binpkgs.sh
+++ b/scripts/obs/gerrit_binpkgs.sh
@@ -1,7 +1,16 @@
 #!/bin/sh -e
 SCRIPTS_OBS_DIR="$(realpath "$(dirname "$0")")"
-
 DISTRO="$1"
+
+error_exit() {
+       echo "---"
+       echo "NOTE: if the build failed because dependencies are outdated, see 
the status here:"
+       echo "https://obs.osmocom.org/project/show/osmocom:master";
+       echo "---"
+       exit 1
+}
+
+
 if [ -z "$DISTRO" ]; then
        echo "usage: gerrit-binpkgs.sh DISTRO"
        echo "examples:"
@@ -33,11 +42,11 @@
        --git-skip-fetch \
        --git-skip-checkout \
        --no-meta \
-       "$PROJECT_NAME"
+       "$PROJECT_NAME" || error_exit

 echo ":: Building the binary packages"
 "$SCRIPTS_OBS_DIR"/build_binpkg.py \
        --docker "$DISTRO" \
-       "$PROJECT_NAME"
+       "$PROJECT_NAME" || error_exit
 
 echo ":: Find binary packages in: $SCRIPTS_OBS_DIR/_temp/binpkgs"

--
To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/38699?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings?usp=email

Gerrit-MessageType: merged
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: I6a2d7c27f6cd8f0e688aefacc819d3760f21e02e
Gerrit-Change-Number: 38699
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>

Reply via email to