Change in ...libosmocore[master]: Drop whitespace after = when parsing LIBVERSION

laforge gerrit-no-reply at
Tue Aug 6 16:21:33 UTC 2019

laforge has submitted this change and it was merged. ( )

Change subject: Drop whitespace after = when parsing LIBVERSION
...................................................................... Drop whitespace after = when parsing LIBVERSION

As a result whitespace ended up in some variables and then command
"expr" was not happy about it.
It was spotted because src/coding/ had some whitespacing.
Since it's the only one, let's drop the whitespace there too to have
similar line in all files.

Change-Id: I33afef5e4ef9eb36de81274533f46598ba9a0edb
M src/coding/
2 files changed, 2 insertions(+), 2 deletions(-)

  Jenkins Builder: Verified
  laforge: Looks good to me, approved

diff --git a/ b/
index 4d4d080..4446da7 100755
--- a/
+++ b/
@@ -47,7 +47,7 @@
 	if [ "z$ALLOW_NO_LIBVERSION_DEB_MATCH" = "z0" ]; then
 		echo "$LIBVERS" | while read -r line; do
-			libversion=$(echo "$line" | cut -d "=" -f 2)
+			libversion=$(echo "$line" | cut -d "=" -f 2 | tr -d "[:space:]")
 			major="$(libversion_to_deb_major "$libversion")"
 			file_matches="$(find "${GIT_TOPDIR}/debian" -name "lib*${major}.install" | wc -l)"
 			if [ "z$file_matches" = "z0" ]; then
diff --git a/src/coding/ b/src/coding/
index c001c13..f47fe45 100644
--- a/src/coding/
+++ b/src/coding/
@@ -1,7 +1,7 @@
 # This is _NOT_ the library release version, it's an API version.
 # Please read Chapter 6 "Library interface versions" of the libtool
 # documentation before making any modification
 	-I"$(top_srcdir)/include" \

To view, visit
To unsubscribe, or for help writing mail filters, visit

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I33afef5e4ef9eb36de81274533f46598ba9a0edb
Gerrit-Change-Number: 15070
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin at>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge at>
Gerrit-Reviewer: osmith <osmith at>
Gerrit-Reviewer: pespin <pespin at>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <>

More information about the gerrit-log mailing list