Change in osmo-ci[master]: docker: only one apt/pip cmd for installing deps

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

pespin gerrit-no-reply at lists.osmocom.org
Fri Oct 11 10:52:13 UTC 2019


pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/15761 )

Change subject: docker: only one apt/pip cmd for installing deps
......................................................................


Patch Set 2: Code-Review+1

(1 comment)

https://gerrit.osmocom.org/c/osmo-ci/+/15761/2/docker/Dockerfile_osmocom_jenkins.amd64 
File docker/Dockerfile_osmocom_jenkins.amd64:

https://gerrit.osmocom.org/c/osmo-ci/+/15761/2/docker/Dockerfile_osmocom_jenkins.amd64@14 
PS2, Line 14: 		asciidoc \
I think it makes sense to write to the right of each line who requires the dependency, specially now that you are removing that info from below when joining all apt install lines together.

For instance:
    liblua5.3-dev \ # osmocom-bb



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

Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: I190e62dd39533b284c93107607dac24ece6d7be6
Gerrit-Change-Number: 15761
Gerrit-PatchSet: 2
Gerrit-Owner: osmith <osmith at sysmocom.de>
Gerrit-Reviewer: laforge <laforge at osmocom.org>
Gerrit-Reviewer: osmith <osmith at sysmocom.de>
Gerrit-Reviewer: pespin <pespin at sysmocom.de>
Gerrit-Comment-Date: Fri, 11 Oct 2019 10:52:13 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20191011/317b4b24/attachment.htm>


More information about the gerrit-log mailing list