osmo-ci[master]: Use stow for dependency management

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/.

Alexander Huemer gerrit-no-reply at lists.osmocom.org
Tue Jun 13 22:44:26 UTC 2017


Patch Set 2:

(4 comments)

https://gerrit.osmocom.org/#/c/2691/2//COMMIT_MSG
Commit Message:

Line 12: has the advantage of letting the build fail if not all location listings
> I find this really hard to read. "Installing dependent libraries in..."? Ra
Done


Line 18: building when dependencies are installed in distinct directories
> what so forth? AFAIK those two are all that is needed? Does it also manage 
Correct, those two are all that is needed. Commit message corrected.
AM_CFLAGS is handled via pkg-config.


https://gerrit.osmocom.org/#/c/2691/2/scripts/osmo-build-dep.sh
File scripts/osmo-build-dep.sh:

Line 51: mkdir -p "$inst/stow"
> $inst is already an empty folder intended as prefix target. Do you really n
I added a comment in the script as justification.


Line 56: STOW_DIR="$inst/stow" stow --restow $project
> I'm curious what this does. Could it get a single line comment?
Done


-- 
To view, visit https://gerrit.osmocom.org/2691
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I8f5012419495a656912b7b71e4f76ce102c6b63a
Gerrit-PatchSet: 2
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Owner: Alexander Huemer <alexander.huemer at xx.vu>
Gerrit-Reviewer: Alexander Huemer <alexander.huemer at xx.vu>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Holger Freyther <holger at freyther.de>
Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: neels <nhofmeyr at sysmocom.de>
Gerrit-HasComments: Yes



More information about the gerrit-log mailing list