Attention is currently required from: osmith, pespin.
laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-iuh/+/30081 )
Change subject: asn1: fix visibility warnings from generated code
......................................................................
Patch Set 2:
(1 comment)
Patchset:
PS2:
> I'm not sure we want to touch the asn files. […]
The asn1 files were already modified quite a bit to make asn1c digest them at all. look at the git commit log of the asn1 subdirectory and you'll find commits like 355d9513c0359ec10de79cc12b6e8a2d4e81e8d8 "Rewrite to avoid information object classes".
So there's no difference here, other than it's 'just' warnings.
What I'm more worried about is that I don't have deep enough ASN.1 and various encoding rules knowledge if this kind of change really doesn't affect the binary encoding/decoding. Only encoding/decoding tests with a reasonably sized body of binary messages would make us notice that, right?
--
To view, visit https://gerrit.osmocom.org/c/osmo-iuh/+/30081
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-iuh
Gerrit-Branch: master
Gerrit-Change-Id: If84445ed2e0df604b581684dcf83f8520b7da84c
Gerrit-Change-Number: 30081
Gerrit-PatchSet: 2
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: laforge <laforge(a)osmocom.org>
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Fri, 11 Nov 2022 22:00:33 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: comment
laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-iuh/+/30080 )
Change subject: move-asn1-header-files: error for wrong asn1c
......................................................................
move-asn1-header-files: error for wrong asn1c
Add a helpful error message if the user doesn't have the expected asn1c
version installed. Link to a new wiki page that explains which branch
one needs to build.
Without this patch, it just complains that HNBAP_Criticality.h can't be
moved.
Change-Id: I11980bf868055bccc05c5338ea330b70022cd874
---
M move-asn1-header-files.sh
1 file changed, 16 insertions(+), 0 deletions(-)
Approvals:
Jenkins Builder: Verified
fixeria: Looks good to me, approved
pespin: Looks good to me, but someone else must approve
laforge: Looks good to me, approved
diff --git a/move-asn1-header-files.sh b/move-asn1-header-files.sh
index ec94e24..7c996b2 100755
--- a/move-asn1-header-files.sh
+++ b/move-asn1-header-files.sh
@@ -12,6 +12,22 @@
include_subdir="$1"
shift
+# Abort if any of the header files are missing
+for i in "$@"; do
+ if ! [ -e "$i" ]; then
+ echo
+ echo "ERROR: file '$i' has not been generated by asn1c!"
+ echo
+ echo "Are you using the Osmocom asn1c fork with the" \
+ "ASN1C_PREFIX feature?"
+ echo
+ echo "More information:"
+ echo "https://osmocom.org/projects/osmohnbgw/wiki/Generate_sources_from_ASN1_files"
+ echo
+ exit 1
+ fi
+done
+
include_dir="$base_dir/include/$include_subdir"
mkdir -p "$include_dir"
echo "$PWD/*.h --> $include_dir"
--
To view, visit https://gerrit.osmocom.org/c/osmo-iuh/+/30080
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-iuh
Gerrit-Branch: master
Gerrit-Change-Id: I11980bf868055bccc05c5338ea330b70022cd874
Gerrit-Change-Number: 30080
Gerrit-PatchSet: 2
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: merged
laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/30091 )
Change subject: update very outdated vty copyright statement
......................................................................
update very outdated vty copyright statement
Change-Id: I9bf6466a6b7f3347453a89ab754c372f6afa920d
---
M src/osmo-bsc/osmo_bsc_main.c
1 file changed, 3 insertions(+), 1 deletion(-)
Approvals:
fixeria: Looks good to me, but someone else must approve
pespin: Looks good to me, approved
Jenkins Builder: Verified
diff --git a/src/osmo-bsc/osmo_bsc_main.c b/src/osmo-bsc/osmo_bsc_main.c
index c85f1c4..91db7be 100644
--- a/src/osmo-bsc/osmo_bsc_main.c
+++ b/src/osmo-bsc/osmo_bsc_main.c
@@ -614,7 +614,9 @@
.copyright =
"Copyright (C) 2008-2018 Harald Welte, Holger Freyther\r\n"
"Contributions by Daniel Willmann, Jan Lübbe, Stefan Schmidt\r\n"
- "Dieter Spaar, Andreas Eversberg, Sylvain Munaut, Neels Hofmeyr\r\n\r\n"
+ "Dieter Spaar, Andreas Eversberg, Sylvain Munaut, Neels Hofmeyr\r\n"
+ "Copyright (C) 2013-2022 sysmocom - s.f.m.c. GmbH\r\n"
+ "\r\n"
"License AGPLv3+: GNU AGPL version 3 or later <http://gnu.org/licenses/agpl-3.0.html>\r\n"
"This is free software: you are free to change and redistribute it.\r\n"
"There is NO WARRANTY, to the extent permitted by law.\r\n",
--
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/30091
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I9bf6466a6b7f3347453a89ab754c372f6afa920d
Gerrit-Change-Number: 30091
Gerrit-PatchSet: 2
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: merged
laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-pcu/+/30089 )
Change subject: update horribly outdated copyright statement on VTY
......................................................................
update horribly outdated copyright statement on VTY
Change-Id: I26cda7826e9d648ea086bb408293e431a9cac206
---
M src/pcu_vty.c
1 file changed, 2 insertions(+), 2 deletions(-)
Approvals:
Jenkins Builder: Verified
fixeria: Looks good to me, but someone else must approve
pespin: Looks good to me, approved
diff --git a/src/pcu_vty.c b/src/pcu_vty.c
index 6b33e29..244a58a 100644
--- a/src/pcu_vty.c
+++ b/src/pcu_vty.c
@@ -1241,8 +1241,8 @@
}
static const char pcu_copyright[] =
- "Copyright (C) 2012 by Ivan Kluchnikov <kluchnikovi(a)gmail.com> and \r\n"
- " Andreas Eversberg <jolly(a)eversberg.eu>\r\n"
+ "Copyright (C) 2012-2013 by Ivan Kluchnikov and Andreas Eversberg\r\n"
+ "Copyright (C) 2013-2022 by sysmocom - s.f.m.c. GmbH\r\n"
"License GNU GPL version 2 or later\r\n"
"This is free software: you are free to change and redistribute it.\r\n"
"There is NO WARRANTY, to the extent permitted by law.\r\n";
--
To view, visit https://gerrit.osmocom.org/c/osmo-pcu/+/30089
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: I26cda7826e9d648ea086bb408293e431a9cac206
Gerrit-Change-Number: 30089
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: merged
laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-bts/+/30090 )
Change subject: update outdated vty copyright statement
......................................................................
update outdated vty copyright statement
Change-Id: Ia12a012c229f883286e96a90132adcc5e8c0c5da
---
M src/common/vty.c
1 file changed, 2 insertions(+), 1 deletion(-)
Approvals:
Jenkins Builder: Verified
fixeria: Looks good to me, but someone else must approve
pespin: Looks good to me, approved
diff --git a/src/common/vty.c b/src/common/vty.c
index 38448b1..a33d719 100644
--- a/src/common/vty.c
+++ b/src/common/vty.c
@@ -149,7 +149,8 @@
}
static const char osmobts_copyright[] =
- "Copyright (C) 2010, 2011 by Harald Welte, Andreas Eversberg and On-Waves\r\n"
+ "Copyright (C) 2010-2011 by Harald Welte, Andreas Eversberg and On-Waves\r\n"
+ "Copyright (C) 2011-2022 by sysmocom - s.f.m.c. GmbH\r\n"
"License AGPLv3+: GNU AGPL version 3 or later <http://gnu.org/licenses/agpl-3.0.html>\r\n"
"This is free software: you are free to change and redistribute it.\r\n"
"There is NO WARRANTY, to the extent permitted by law.\r\n";
--
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/30090
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: Ia12a012c229f883286e96a90132adcc5e8c0c5da
Gerrit-Change-Number: 30090
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: merged