Attention is currently required from: laforge, fixeria.
Hoernchen has posted comments on this change. (
https://gerrit.osmocom.org/c/osmo-ci/+/30791 )
Change subject: coverity: git pull --recursive / update submodules
......................................................................
Patch Set 2:
(1 comment)
File coverity/prepare_source_Osmocom.sh:
https://gerrit.osmocom.org/c/osmo-ci/+/30791/comment/a472d15f_7c951552
PS1, Line 55: && git submodule update --recursive --remote
I am not certain either, my experience with
git-submodules is extremely limited... […]
As the related commit message states it
is only for local testing anyway, I don't know what the user wants to do here, this
also applies to the cleaning below, so I'd wouldn't touch it, which is why I
didn't modify this part in my own patch that i just abandoned because this patch has
more lines and is therefore better.
--
To view, visit
https://gerrit.osmocom.org/c/osmo-ci/+/30791
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: I876fa0152268bd7a43ca4c452d75c26ebadf7a6e
Gerrit-Change-Number: 30791
Gerrit-PatchSet: 2
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 28 Dec 2022 15:21:05 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: laforge <laforge(a)osmocom.org>
Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-MessageType: comment