osmith has uploaded this change for review. ( 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, 17 insertions(+), 0 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-iuh refs/changes/80/30080/1 diff --git a/move-asn1-header-files.sh b/move-asn1-header-files.sh index ec94e24..a0e57da 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" @@ -27,6 +43,7 @@ } move_headers() { + echo "pwd: $PWD" echo mv $@ "$include_dir/" mv $@ "$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: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-MessageType: newchange