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

Unified Diff: tools/create_sdk.py

Issue 10583037: Enables building dart_analyzer on windows and packaging it in the SDK (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 8 years, 6 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 | « dart.gyp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/create_sdk.py
diff --git a/tools/create_sdk.py b/tools/create_sdk.py
index aeba302c1d48ca060a1d6378923b296270809d13..4078118b78f698e72fe0d484c4a685fe3e84cfe9 100755
--- a/tools/create_sdk.py
+++ b/tools/create_sdk.py
@@ -95,12 +95,6 @@ def Copy(src, dest):
copyfile(src, dest)
copymode(src, dest)
-# TODO(zundel): this excludes the analyzer from the sdk build until builders
-# have all prerequisite software installed. Also update dart.gyp.
-def ShouldCopyAnalyzer():
- os = utils.GuessOS();
- return os == 'linux' or os == 'macos'
-
def CopyShellScript(src_file, dest_dir):
'''Copies a shell/batch script to the given destination directory. Handles
@@ -198,14 +192,12 @@ def Main(argv):
copyfile(dart_src_binary, dart_dest_binary)
copymode(dart_src_binary, dart_dest_binary)
- if ShouldCopyAnalyzer():
- # Copy analyzer into sdk/bin
- ANALYZER_HOME = join(HOME, build_dir, 'analyzer')
- dart_analyzer_src_binary = join(ANALYZER_HOME, 'bin', 'dart_analyzer')
- dart_analyzer_dest_binary = join(BIN,
- 'dart_analyzer' + analyzer_file_extension)
- copyfile(dart_analyzer_src_binary, dart_analyzer_dest_binary)
- copymode(dart_analyzer_src_binary, dart_analyzer_dest_binary)
+ ANALYZER_HOME = join(HOME, build_dir, 'analyzer')
+ dart_analyzer_src_binary = join(ANALYZER_HOME, 'bin', 'dart_analyzer')
+ dart_analyzer_dest_binary = join(BIN,
+ 'dart_analyzer' + analyzer_file_extension)
+ copyfile(dart_analyzer_src_binary, dart_analyzer_dest_binary)
+ copymode(dart_analyzer_src_binary, dart_analyzer_dest_binary)
# Create pub shell script.
pub_src_script = join(HOME, 'utils', 'pub', 'sdk', 'pub')
@@ -429,25 +421,24 @@ def Main(argv):
UTIL = join(SDK_tmp, 'util')
os.makedirs(UTIL)
- if ShouldCopyAnalyzer():
- # Create and copy Analyzer library into 'util'
- ANALYZER_DEST = join(UTIL, 'analyzer')
- os.makedirs(ANALYZER_DEST)
+ # Create and copy Analyzer library into 'util'
+ ANALYZER_DEST = join(UTIL, 'analyzer')
+ os.makedirs(ANALYZER_DEST)
- analyzer_src_jar = join(ANALYZER_HOME, 'util', 'analyzer',
+ analyzer_src_jar = join(ANALYZER_HOME, 'util', 'analyzer',
'dart_analyzer.jar')
- analyzer_dest_jar = join(ANALYZER_DEST, 'dart_analyzer.jar')
- copyfile(analyzer_src_jar, analyzer_dest_jar)
-
- jarsToCopy = [ join("args4j", "2.0.12", "args4j-2.0.12.jar"),
- join("guava", "r09", "guava-r09.jar"),
- join("json", "r2_20080312", "json.jar") ]
- for jarToCopy in jarsToCopy:
- dest_dir = join (ANALYZER_DEST, os.path.dirname(jarToCopy))
- os.makedirs(dest_dir)
- dest_file = join (ANALYZER_DEST, jarToCopy)
- src_file = join(ANALYZER_HOME, 'util', 'analyzer', jarToCopy)
- copyfile(src_file, dest_file)
+ analyzer_dest_jar = join(ANALYZER_DEST, 'dart_analyzer.jar')
+ copyfile(analyzer_src_jar, analyzer_dest_jar)
+
+ jarsToCopy = [ join("args4j", "2.0.12", "args4j-2.0.12.jar"),
+ join("guava", "r09", "guava-r09.jar"),
+ join("json", "r2_20080312", "json.jar") ]
+ for jarToCopy in jarsToCopy:
+ dest_dir = join (ANALYZER_DEST, os.path.dirname(jarToCopy))
+ os.makedirs(dest_dir)
+ dest_file = join (ANALYZER_DEST, jarToCopy)
+ src_file = join(ANALYZER_HOME, 'util', 'analyzer', jarToCopy)
+ copyfile(src_file, dest_file)
# Create and populate util/pub.
pub_src_dir = join(HOME, 'utils', 'pub')
« no previous file with comments | « dart.gyp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698