Attention is currently required from: pespin.
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-cbc/+/28601
to look at the new patch set (#3).
Change subject: Add unit tests for sbcap
......................................................................
Add unit tests for sbcap
This commit adds some minimum unit tests to validate encoding and
decoding of SBc-AP ASN.1 APER format is working properly.
Related: OS#4945
Change-Id: I5a26d008b7282da637c3781ae77435c6a69cf65d
---
M configure.ac
M tests/Makefile.am
A tests/sbcap/Makefile.am
A tests/sbcap/sbcap_test.c
A tests/sbcap/sbcap_test.ok
A tests/testsuite.at
6 files changed, 346 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-cbc refs/changes/01/28601/3
--
To view, visit https://gerrit.osmocom.org/c/osmo-cbc/+/28601
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-cbc
Gerrit-Branch: master
Gerrit-Change-Id: I5a26d008b7282da637c3781ae77435c6a69cf65d
Gerrit-Change-Number: 28601
Gerrit-PatchSet: 3
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: newpatchset
Attention is currently required from: pespin.
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-cbc/+/28601
to look at the new patch set (#2).
Change subject: Add unit tests for sbcap
......................................................................
Add unit tests for sbcap
This commit adds some minimum unit tests to validate encoding and
decoding of SBc-AP ASN.1 APER format is working properly.
Related: OS#4945
Change-Id: I5a26d008b7282da637c3781ae77435c6a69cf65d
---
M configure.ac
M tests/Makefile.am
A tests/sbcap/Makefile.am
A tests/sbcap/sbcap_test.c
A tests/sbcap/sbcap_test.ok
A tests/testsuite.at
6 files changed, 346 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-cbc refs/changes/01/28601/2
--
To view, visit https://gerrit.osmocom.org/c/osmo-cbc/+/28601
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-cbc
Gerrit-Branch: master
Gerrit-Change-Id: I5a26d008b7282da637c3781ae77435c6a69cf65d
Gerrit-Change-Number: 28601
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: newpatchset
Attention is currently required from: osmith, fixeria.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/28606 )
Change subject: lint: ignore *.asn, *.cc
......................................................................
Patch Set 1:
(1 comment)
File lint/checkpatch/checkpatch_osmo.sh:
https://gerrit.osmocom.org/c/osmo-ci/+/28606/comment/388fc96a_183667b9
PS1, Line 8: asn|cc
> I don't think this is the right approach. We may have hand-written *.asn and *. […]
I definetly wouldn't blacklist all .cc by default. I don't care that much for asn files.
--
To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/28606
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: I3c00bde31f11d40719f014fa1eb3162930984c56
Gerrit-Change-Number: 28606
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Comment-Date: Thu, 14 Jul 2022 14:38:54 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-MessageType: comment
Attention is currently required from: osmith.
fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/28606 )
Change subject: lint: ignore *.asn, *.cc
......................................................................
Patch Set 1:
(1 comment)
File lint/checkpatch/checkpatch_osmo.sh:
https://gerrit.osmocom.org/c/osmo-ci/+/28606/comment/73aab762_b81be883
PS1, Line 8: asn|cc
I don't think this is the right approach. We may have hand-written *.asn and *.cc files in some projects and they'll be ignored by the linter.
--
To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/28606
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: I3c00bde31f11d40719f014fa1eb3162930984c56
Gerrit-Change-Number: 28606
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: osmith <osmith(a)sysmocom.de>
Gerrit-Comment-Date: Thu, 14 Jul 2022 14:36:32 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: pespin.
fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/28603 )
Change subject: Support for SBc-AP protocol as used on the MME-CBC interface
......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
Also, where the *.asn files are coming from? Please add a link/reference to the commit message.
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/28603
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: Ia6743e0a3e7974a5f2dd3ecf74ec331f646f6bc2
Gerrit-Change-Number: 28603
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Thu, 14 Jul 2022 12:53:41 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment