fixeria has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/42333?usp=email )
Change subject: library/ngap: do not link against libfftranscode
......................................................................
library/ngap: do not link against libfftranscode
TITAN has built-in ASN.1 PER codec since version 10.0.0.
We're already using it for NGAP, so libfftranscode is no longer needed.
Change-Id: I7ea63993278a5e0ca22fc4217c2c3c5f8f9dc0a6
Related: OS#6490
---
M library/ngap/regen_makefile.sh
1 file changed, 0 insertions(+), 4 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/33/42333/1
diff --git a/library/ngap/regen_makefile.sh b/library/ngap/regen_makefile.sh
index d630ab9..6a607bf 100755
--- a/library/ngap/regen_makefile.sh
+++ b/library/ngap/regen_makefile.sh
@@ -3,7 +3,3 @@
FILES="*.asn *.ttcn NGAP_EncDec.cc"
../../_buildsystem/regen-makefile.sh $FILES
-
-sed -i -e '/^CPPFLAGS/ s/$/ `pkg-config --cflags libfftranscode`/' Makefile
-sed -i -e '/^LDFLAGS/ s/$/ `pkg-config --libs libfftranscode`/' Makefile
-sed -i -e '/^LINUX_LIBS/ s/$/ `pkg-config --libs libfftranscode`/' Makefile
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/42333?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newchange
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I7ea63993278a5e0ca22fc4217c2c3c5f8f9dc0a6
Gerrit-Change-Number: 42333
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Hoernchen has posted comments on this change by Hoernchen. ( https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/42330?usp=email )
Change subject: fw source: force reformat once and for all
......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
This is the result of clang-format-20 --verbose -style=file:.clang-format -i {sysmoOCTSIM,ccid_common}/*{.c,.h} and the linter disagrees, so at this point I don't even know what rules are being enforced, apparently not the rules specified by the .clang-format file.
--
To view, visit https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/42330?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-ccid-firmware
Gerrit-Branch: master
Gerrit-Change-Id: Iacc086bb566551225e7a21b639a1ad2ec257484f
Gerrit-Change-Number: 42330
Gerrit-PatchSet: 1
Gerrit-Owner: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Comment-Date: Thu, 12 Mar 2026 16:40:30 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Attention is currently required from: Hoernchen.
Hoernchen has posted comments on this change by Hoernchen. ( https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/42330?usp=email )
Change subject: fw source: force reformat once and for all
......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
Clearly the formatting rules expected by the linter disagree a lot with all of the existing code. Either the rules need to be adjusted, or the code, either way, one of these has to happen to put this topic to rest after years of formatting misadventures.
--
To view, visit https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/42330?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-ccid-firmware
Gerrit-Branch: master
Gerrit-Change-Id: Iacc086bb566551225e7a21b639a1ad2ec257484f
Gerrit-Change-Number: 42330
Gerrit-PatchSet: 1
Gerrit-Owner: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Attention: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Comment-Date: Thu, 12 Mar 2026 16:23:11 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/42329?usp=email
to look at the new patch set (#2).
The following approvals got outdated and were removed:
Verified+1 by Jenkins Builder
Change subject: 5gc: parametrize RequestType in f_pdu_sess_establish()
......................................................................
5gc: parametrize RequestType in f_pdu_sess_establish()
Change-Id: I69d6be5e581565720a21c719b0f982eb31ec2adf
---
M 5gc/C5G_Tests.ttcn
M 5gc/ConnHdlr.ttcn
M library/NG_NAS_Osmo_Types.ttcn
3 files changed, 19 insertions(+), 9 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/29/42329/2
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/42329?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I69d6be5e581565720a21c719b0f982eb31ec2adf
Gerrit-Change-Number: 42329
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Attention is currently required from: laforge.
Hoernchen has posted comments on this change by Hoernchen. ( https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/42306?usp=email )
Change subject: ccid: properly emit wait time ext messages
......................................................................
Patch Set 3:
(1 comment)
File ccid_common/iso7816_fsm.h:
https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/42306/comment/e523903a_3a… :
PS2, Line 25: ISO7816_E_RX_SINGLE, /*!< single-byte data received on UART */
> I don't understant why it's so hard to first reformat the code, and then add your changes on top of […]
yes, really interesting, i wonder why that is? interestingly I rarely see any other developer even submitting any kind of "formatting first then changes" commits for review in the first place, maybe because no one touches those code parts with artisanal formatting that are impossible to modify without reformatting or adjusting the linter? Its really weird how I'm apaprently supposed to take shit for touching these parts, as usual.
--
To view, visit https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/42306?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-ccid-firmware
Gerrit-Branch: master
Gerrit-Change-Id: Ib69483d453a0e5ebb1bc1885a8f78790a0f10d70
Gerrit-Change-Number: 42306
Gerrit-PatchSet: 3
Gerrit-Owner: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Comment-Date: Thu, 12 Mar 2026 15:34:42 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Hoernchen <ewild(a)sysmocom.de>
Comment-In-Reply-To: laforge <laforge(a)osmocom.org>