Installing ChromeOS font on developer Linux machine

build/install-build-deps.sh

Index: build/install-build-deps.sh
diff --git a/build/install-build-deps.sh b/build/install-build-deps.sh
index 4cb5165a0c0fb761dc6ce361ef9468ff44dac04f..474f8f869bed5780993918bd99bc4dc9c71a5477 100755
--- a/build/install-build-deps.sh
+++ b/build/install-build-deps.sh
@@ -14,6 +14,7 @@ usage() {
   echo "--[no-]syms: enable or disable installation of debugging symbols"
   echo "--[no-]lib32: enable or disable installation of 32 bit libraries"
   echo "--[no-]arm: enable or disable installation of arm cross toolchain"
+  echo "--[no-]chromeos-font: enable or disable installation of ChromeOS font"

[Daniel Erat, 2013/02/21 20:38:03]

nit: "chromeos-fonts" would probably be better, in case more get added later

[sschmitz, 2013/02/21 21:47:44]

Done.

   echo "--no-prompt: silently select standard options/defaults"
   echo "Script will prompt interactively if options not given."
   exit 1
@@ -28,6 +29,8 @@ do
   --no-lib32)               do_inst_lib32=0;;
   --arm)                    do_inst_arm=1;;
   --no-arm)                 do_inst_arm=0;;
+  --chromeos-font)          do_inst_chromeos_font=1;;
+  --no-chromeos-font)       do_inst_chromeos_font=0;;
   --no-prompt)              do_default=1
                             do_quietly="-qq --assume-yes"
     ;;
@@ -201,6 +204,14 @@ else
   dbg_list=
 fi
 
+# Install the ChromeOS default font.
+if test "$do_inst_chromeos_font" != "0"; then

[Daniel Erat, 2013/02/21 20:38:03]

nit: echo a message like "Installing Chrome OS fonts.", since that seems to
match what's done for other options (or maybe this isn't necessary, since the
script looks like it outputs a similar message)

[sschmitz, 2013/02/21 21:47:44]

For symmetry, I will add a message. That will add context info for errors. In
addition the script outputs one (of two) messages when successful. They give
more info:

Chrome OS fonts already up-to-date in /usr/local/share/fonts.

OR:

Installing Chrome OS fonts to /usr/local/share/fonts.

Done

+  dir=`echo $0 | sed -r -e 's/\/[^/]+$//'`
+  sudo $dir/linux/install-chromeos-font.py
+else
+  echo "Skipping installation of ChromeOS font."

[Daniel Erat, 2013/02/21 20:38:03]

nit: s/ChromeOS/Chrome OS/, s/font/fonts/

[sschmitz, 2013/02/21 21:47:44]

Done.

+fi
+
 # When cross building for arm on 64-bit systems the host binaries
 # that are part of v8 need to be compiled with -m32 which means
 # that basic multilib support is needed.