Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3486)

Unified Diff: build/linux/install-chromeos-fonts.py

Issue 666413002: Update version of fonts for install-chromeos-fonts.py. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/linux/install-chromeos-fonts.py
diff --git a/build/linux/install-chromeos-fonts.py b/build/linux/install-chromeos-fonts.py
index 98c3a570d139f2a21c21bf0e08c9f9c4278898fd..a24adc920ce10a575d9c4c4e560e13bdef9e9723 100755
--- a/build/linux/install-chromeos-fonts.py
+++ b/build/linux/install-chromeos-fonts.py
@@ -12,16 +12,12 @@ import shutil
import subprocess
import sys
-URL_PREFIX = 'https://commondatastorage.googleapis.com'
-URL_DIR = 'chromeos-localmirror/distfiles'
-URL_FILE = 'notofonts-20121206.tar.gz'
+# Taken from the media-fonts/notofonts ebuild in chromiumos-overlay.
+VERSION = '20140815'
+URL = ('https://commondatastorage.googleapis.com/chromeos-localmirror/'
+ 'distfiles/notofonts-%s.tar.bz2') % (VERSION)
FONTS_DIR = '/usr/local/share/fonts'
-# The URL matches the URL in the ebuild script in chromiumos. See:
-# /path/to/chromiumos/src/
-# third_party/chromiumos-overlay/media-fonts/notofonts/
-# notofonts-20121206.ebuild
-
def main(args):
if not sys.platform.startswith('linux'):
print "Error: %s must be run on Linux." % __file__
@@ -37,12 +33,10 @@ def main(args):
dest_dir = os.path.join(FONTS_DIR, 'chromeos')
- url = "%s/%s/%s" % (URL_PREFIX, URL_DIR, URL_FILE)
-
stamp = os.path.join(dest_dir, ".stamp02")
if os.path.exists(stamp):
with open(stamp) as s:
- if s.read() == url:
+ if s.read() == URL:
print "Chrome OS fonts already up-to-date in %s." % dest_dir
return 0
@@ -52,8 +46,8 @@ def main(args):
os.chmod(dest_dir, 0755)
print "Installing Chrome OS fonts to %s." % dest_dir
- tarball = os.path.join(dest_dir, URL_FILE)
- subprocess.check_call(['curl', '-L', url, '-o', tarball])
+ tarball = os.path.join(dest_dir, os.path.basename(URL))
+ subprocess.check_call(['curl', '-L', URL, '-o', tarball])
subprocess.check_call(['tar', '--no-same-owner', '--no-same-permissions',
'-xf', tarball, '-C', dest_dir])
os.remove(tarball)
@@ -65,7 +59,7 @@ def main(args):
s.write("Script: %s\n" % __file__)
with open(stamp, 'w') as s:
- s.write(url)
+ s.write(URL)
for base, dirs, files in os.walk(dest_dir):
for dir in dirs:
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698