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

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

 #!/bin/bash -e
 
 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # 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
 debchange --create \
   --package chrome-remote-desktop \
   --distribution lucid \
   --newversion "$version_full" \
   "New Debian package"
 
 dpkg-buildpackage -b -us -uc