Commit 1c480ab6 authored by sschmitz@chromium.org's avatar sschmitz@chromium.org

Installing Chrome OS fonts fails when repo is on filer

install-build-deps.sh runs "sudo /path/to/src/build/linux/install-chromeos-fonts.py".
If the repo is on a filer, sudo fails because root is not allowed to access files on
a filer. However, installing a Repo on a Filer is not recommended.
Solution:
install-build-deps.sh attempts to detect that the Repo is not on a Filer and
in that case prints a warning with instructions.

Note: install-build-deps.sh exits immediately on failure because it uses "bash -e".

BUG=225543
R=derat@chromium.org
TEST=manual

Review URL: https://chromiumcodereview.appspot.com/13532003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@192485 0039d316-1c4b-4281-b951-d872f2087c98
parent 02e35f29
......@@ -206,9 +206,21 @@ fi
# Install the Chrome OS default fonts.
if test "$do_inst_chromeos_fonts" != "0"; then
echo
echo "Installing Chrome OS fonts."
dir=`echo $0 | sed -r -e 's/\/[^/]+$//'`
sudo $dir/linux/install-chromeos-fonts.py
if ! sudo $dir/linux/install-chromeos-fonts.py; then
echo "ERROR: The installation of the Chrome OS default fonts failed."
if [ `stat -f -c %T $dir` == "nfs" ]; then
echo "The reason is that your repo is installed on a remote file system."
else
echo "This is expected if your repo is installed on a remote file system."
fi
echo "It is recommended to install your repo on a local file system."
echo "You can skip the installation of the Chrome OS default founts with"
echo "the command line option: --no-chromeos-fonts."
exit 1
fi
else
echo "Skipping installation of Chrome OS fonts."
fi
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment