laforge submitted this change.

View Change

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
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(-)

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 change 30080. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmo-iuh
Gerrit-Branch: master
Gerrit-Change-Id: I11980bf868055bccc05c5338ea330b70022cd874
Gerrit-Change-Number: 30080
Gerrit-PatchSet: 2
Gerrit-Owner: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged