Update build-deb.sh to automatically detect version number.

remoting/host/installer/linux/build-deb.sh

Index: remoting/host/installer/linux/build-deb.sh
diff --git a/remoting/host/installer/linux/build-deb.sh b/remoting/host/installer/linux/build-deb.sh
index 520477622b757e8e67a82d880e21bb118604a9ad..14231cb3e32e17a1a8c326c0e4cbe448bb99205f 100755
--- a/remoting/host/installer/linux/build-deb.sh
+++ b/remoting/host/installer/linux/build-deb.sh
@@ -5,10 +5,23 @@
 # found in the LICENSE file.
 
 if [[ -z "$version_full" ]]; then
-  echo 'Error: $version_full not set'
+  src_root=./../../../..
+  version_helper=$src_root/chrome/tools/build/version.py
+  version_base=$($version_helper -f $src_root/remoting/VERSION \
+                 -t "@MAJOR@.@MINOR@")
+  version_build=$($version_helper -f $src_root/chrome/VERSION \
+                  -t "@BUILD@.@PATCH@")
+  version_full="$version_base.$version_build"
+fi
+
+if !(echo $version_full | grep -E \

[Lambros, 2012/08/27 21:39:33]

Optional: maybe move this check into the first if.. block, in case we ever want
to build with a tweaked version string?

[Sergey Ulanov, 2012/08/28 00:41:07]

It's intentionally outside of the previous if to make sure version is correct
even when specified externally. We can always change this regex if we ever need
to support different version string format.

+     "^[[:digit:]]+.[[:digit:]]+.[[:digit:]]+.[[:digit:]]+$" > /dev/null); then

[Jamie, 2012/08/27 21:03:31]

Strictly speaking, you should escape the . characters. Also, FYI, you can
express this more succinctly with the =~ operator:

[[ "$version_full" =~ ^[[:digit:]]+\.[[:digit:]]+\.[[:digit:]]+\.[[:digit:]]+$
]]

I'm not sure it's any more readable, however.

+  echo "Error: Invalid \$version_full value: $version_full" >&2
   exit 1
 fi
 
+echo "Building version $version_full"
+
 # Create a fresh debian/changelog.
 export DEBEMAIL="The Chromium Authors <chromium-dev@chromium.org>"
 rm -f debian/changelog