fixeria submitted this change.

View Change


Approvals: pespin: Looks good to me, approved osmith: Looks good to me, but someone else must approve Jenkins Builder: Verified
contrib/jenkins.sh: uncomment verify_value_string_arrays_are_terminated.py

Change-Id: Iad9a6546fa7e2a654ed3217c2a875357ef5cc804
---
M contrib/jenkins.sh
1 file changed, 12 insertions(+), 2 deletions(-)

diff --git a/contrib/jenkins.sh b/contrib/jenkins.sh
index 5cc3a67..3127873 100755
--- a/contrib/jenkins.sh
+++ b/contrib/jenkins.sh
@@ -17,8 +17,9 @@

mkdir "$deps" || true

-# TODO: ask whether fail is expected, because osmocom-bb build succeeds?
-#"$deps"/libosmocore/contrib/verify_value_string_arrays_are_terminated.py $(find . -name "*.[hc]")
+# exclude ancient local copy of libosmocore.git
+verify_value_string_arrays_are_terminated.py \
+ $(find . -path ./src/shared/libosmocore -prune -o -name '*.[hc]' -print)

export PKG_CONFIG_PATH="$inst/lib/pkgconfig:$PKG_CONFIG_PATH"
export LD_LIBRARY_PATH="$inst/lib"

To view, visit change 32747. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: Iad9a6546fa7e2a654ed3217c2a875357ef5cc804
Gerrit-Change-Number: 32747
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged