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

Unified Diff: bin/sync-and-gyp

Issue 1425593011: bin/sync-and-gyp: sh->py (make more cross-platform) (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: 2015-11-09 (Monday) 11:13:02 EST Created 5 years, 1 month 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: bin/sync-and-gyp
diff --git a/bin/sync-and-gyp b/bin/sync-and-gyp
index c944079938c771e5b85c8b548e768ed4b63c5545..80dead97583504399f7cbe32b5692482eba1dbd7 100755
--- a/bin/sync-and-gyp
+++ b/bin/sync-and-gyp
@@ -1,58 +1,88 @@
-#!/bin/sh
+#!/usr/bin/env python
# Copyright 2015 Google Inc.
#
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.
-# This script will update Skia's dependenciess as necessary and run
+# This script will update Skia's dependencies as necessary and run
# gyp if needed.
-# Depends on: Posix-compliant shell, Python, and Git.
+# Depends on: Python, and Git.
#
# Example usage:
#
# git clone https://skia.googlesource.com/skia
# cd skia
-# bin/sync-and-gyp
+# python bin/sync-and-gyp
# ninja -C out/Debug && out/Debug/dm
#
# Once changes are made to DEPS or gyp/ or the source, call:
#
-# bin/sync-and-gyp
-
-if [ "$SKIA_OUT" ]; then
- mkdir -p "$SKIA_OUT" || exit
- # get non-relative path of $SKIA_OUT before changing directory.
- SKIA_OUT="$(cd "$SKIA_OUT"; pwd)"
-fi
-
-cd "$(dirname "$0")/.."
-
-if ! [ -f DEPS ]; then
- echo DEPS file missing >&2
- exit 1
-fi
-
-GIT_SYNC_DEPS_QUIET=1 python tools/git-sync-deps || exit
-
-catifexists() { if [ -f "$1" ]; then cat "$1"; fi; }
-
-gyp_hasher() {
- {
- echo "$CC"
- echo "$CXX"
- echo "$GYP_GENERATORS"
- echo "$GYP_DEFINES"
- find gyp -type f -print -exec git hash-object {} \;
- find bench gm tests -name '*.c*' | LANG= sort
- } | git hash-object --stdin
-}
-
-: ${SKIA_OUT:=out}
-GYP_HASH=$(gyp_hasher)
-HASH_PATH="${SKIA_OUT}/gyp_hash"
-if [ "$GYP_HASH" != "$(catifexists "$HASH_PATH")" ]; then
- python ./gyp_skia || exit
- echo "$GYP_HASH" > "$HASH_PATH"
-fi
+# python bin/sync-and-gyp
+
+import fnmatch
+import hashlib
+import subprocess
+import os
+
+skia_dir = os.path.join(os.path.dirname(__file__), os.pardir)
+
+skia_out = os.environ.get("SKIA_OUT")
+if skia_out:
+ skia_out = os.path.abspath(skia_out)
+ hash_path = os.path.join(skia_out, 'gyp_hash')
+else:
+ hash_path = os.path.join('out', 'gyp_hash')
+
+os.chdir(skia_dir)
+
+if not os.path.isfile('DEPS'):
+ sys.stderr.write('DEPS file missing')
+ exit(1)
+
+env = os.environ.copy()
+env["GIT_SYNC_DEPS_QUIET"] = "1"
+subprocess.call(['python', 'tools/git-sync-deps'], env=env)
+
+hasher = hashlib.sha1()
+
+for var in ['AR', 'AR_host', 'AR_target',
+ 'CC', 'CC_host', 'CC_target',
+ 'CFLAGS', 'CFLAGS_host',
+ 'CPPFLAGS', 'CPPFLAGS_host',
+ 'CXX', 'CXX_host', 'CXX_target',
+ 'GYP_DEFINES', 'GYP_GENERATORS',
+ 'NM', 'NM_host', 'NM_target',
+ 'READELF', 'READELF_host', 'READELF_target']:
+ hasher.update(os.environ.get(var, '') + '\n')
+
+def listfiles(folder, matchfilter):
+ for root, folders, files in os.walk(folder):
+ for filename in files:
+ if fnmatch.fnmatch(filename, matchfilter):
+ yield os.path.join(root, filename)
+
+for filename in sorted(listfiles('gyp', '*')):
+ with open(filename, 'r') as f:
+ hasher.update(f.read())
+
+for dir in ['bench', 'gm', 'tests']:
+ for filename in sorted(listfiles(dir, '*.c*')):
+ hasher.update(filename + '\n')
+
+gyp_hash = hasher.hexdigest()
+
+def cat_if_exists(path):
+ if os.path.exists(path):
+ with open(path, 'r') as f:
+ return f.read()
+ return ''
+
+if cat_if_exists(hash_path).strip() != gyp_hash:
+ env = os.environ.copy()
+ if skia_out:
+ env['SKIA_OUT'] = skia_out
+ subprocess.call(['python', './gyp_skia'], env=env)
+ with open(hash_path, 'w') as o:
+ o.write(gyp_hash)
« 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