Attention is currently required from: laforge, osmith. pespin has posted comments on this change by osmith. ( https://gerrit.osmocom.org/c/osmo-ci/+/38698?usp=email )
Change subject: old-tags-against-master: ignore last osmo-bsc tags ...................................................................... Patch Set 2: (1 comment) Patchset: PS2: > We should not have "unavoidable" API breaks in libosmo-sigtran. Something is > wrong here. […] It's simple: I already fixed all users of libosmo-sigtran to avoid using structs. Hence, structs can finally (it should have been like that since the start) be made private. This means I can now modify structs inside the library, which will be needed to implement AS loadsharing. This also means anyone, in the future, can add new features without breaking ABI/API again. So, imho, here we can accept that broken user code accessing structs is not going to build with newer versions of the library without applying a small patchset (already available in master and/or gerrit), or keep breaking ABI again and again because we don't want to fix the root problem, also in the process making my time 10x times more difficult to implement the features. I'm the first one willing to keep ABI stable, but if we want to do that then we need to reasonable and stop making tons of stuff public-first before there's even a known user of those APIs/structs. Otherwise all this is nonsense imho. BTW, this is still happening with newer libraries like libosmo-pfcp, despite I think I'm raising the topic at review time here and there. -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/38698?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: Iac7e6ea7815dd2c3eb9c2636c0caa661161654ed Gerrit-Change-Number: 38698 Gerrit-PatchSet: 2 Gerrit-Owner: osmith <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <[email protected]> Gerrit-Reviewer: laforge <[email protected]> Gerrit-Reviewer: pespin <[email protected]> Gerrit-Attention: osmith <[email protected]> Gerrit-Attention: laforge <[email protected]> Gerrit-Comment-Date: Sun, 10 Nov 2024 22:24:50 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: laforge <[email protected]>
