Attention is currently required from: fixeria.
osmith has posted comments on this change. (
https://gerrit.osmocom.org/c/gapk/+/34892?usp=email )
Change subject: dist: exclude libgsmhr files downloaded by fetch_sources.py
......................................................................
Patch Set 1:
(1 comment)
File libgsmhr/Makefile.am:
https://gerrit.osmocom.org/c/gapk/+/34892/comment/e743eb8e_54f4866d
PS1, Line 39: -rm -rf $(distdir)/$(REFSRC_PATH)
I think removing the sources in a dist-hook is a good idea, so they don't end up in
the archive by accident if they were already downloaded.
However downloading the sources first even if we don't intend to package them, and
then remove them, seems like an unclean solution. Can't we additionally change it so
they don't get downloaded in the first place unless --enable-gsmhr is set?
--
To view, visit
https://gerrit.osmocom.org/c/gapk/+/34892?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: gapk
Gerrit-Branch: master
Gerrit-Change-Id: I66e31dec37e53bf1a8c7df948fd9316e1467752c
Gerrit-Change-Number: 34892
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 25 Oct 2023 10:56:54 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment