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 <vyanits...@sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <lafo...@osmocom.org> Gerrit-Reviewer: osmith <osm...@sysmocom.de> Gerrit-Attention: osmith <osm...@sysmocom.de> Gerrit-Attention: laforge <lafo...@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 <osm...@sysmocom.de> Comment-In-Reply-To: laforge <lafo...@osmocom.org> Gerrit-MessageType: comment