osmith has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-ci/+/32683 )

Change subject: obs: build_binpkg: ubuntu: use manuals container
......................................................................

obs: build_binpkg: ubuntu: use manuals container

Use the optimization of only installing osmo-gsm-manuals-dev and all its
dependencies once not only with debian, but also with ubuntu.

Change-Id: Ic9fef9f10b4384538e7a73479bf57a9b737a9a55
---
M scripts/obs/build_binpkg.py
1 file changed, 13 insertions(+), 1 deletion(-)

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




diff --git a/scripts/obs/build_binpkg.py b/scripts/obs/build_binpkg.py
index 37dc6be..67a4bec 100755
--- a/scripts/obs/build_binpkg.py
+++ b/scripts/obs/build_binpkg.py
@@ -85,7 +85,7 @@

         # Optimization: use docker container with osmo-gsm-manuals-dev already
         # installed if it is in build depends
-        if distro.startswith("debian:") \
+        if env["PACKAGEFORMAT"] == "deb" \
                 and lib.srcpkg.requires_osmo_gsm_manuals_dev(args.package):
             image_type += "_manuals"


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

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

Reply via email to