Attention is currently required from: laforge, osmith.
fixeria has posted comments on this change. (
https://gerrit.osmocom.org/c/osmo-ci/+/35291?usp=email )
Change subject: Revert "coverity: build osmo-trx with USRP1 support"
......................................................................
Patch Set 1:
(1 comment)
Commit Message:
https://gerrit.osmocom.org/c/osmo-ci/+/35291/comment/0f9c2b16_35b988a2
PS1, Line 13: CIDs in Coverity's defect browser, which nobody is going to fix...
There are many features in osmocom that are rarely
tested. […]
FYI: there used to be a Component for `libusrp`, but I removed it
yesterday in hope that it would also remove all the related CIDs. But now they show up as
`Other`. I did some more research today, and I believe the right way to fix this would be
to re-create a component, but setting `Ignore=yes`. I just did that, and yay, the related
defects are all gone now. This means we still check specific parts of `osmo-trx-usrp`, but
not `libusrp` itself anymore. I think we can abandon this patch?
--
To view, visit
https://gerrit.osmocom.org/c/osmo-ci/+/35291?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: I5f47bbcc1388fd7bbe6830481e6a8abcdc582029
Gerrit-Change-Number: 35291
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Comment-Date: Mon, 11 Dec 2023 10:02:15 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: osmith <osmith(a)sysmocom.de>
Comment-In-Reply-To: laforge <laforge(a)osmocom.org>
Gerrit-MessageType: comment