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

Unified Diff: dart/tools/make_links.py

Issue 11414056: Pass package-root option to dart2js in browser tests. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge
Patch Set: Correctly compute relative source on Unix Created 8 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 | dart/tools/testing/dart/test_suite.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: dart/tools/make_links.py
diff --git a/dart/tools/make_links.py b/dart/tools/make_links.py
index 6a410707815f73e8622154c7ed1739a8b3a9a679..3f54004531a43bc8f4348ca5ef44b1c3ca62c63d 100644
--- a/dart/tools/make_links.py
+++ b/dart/tools/make_links.py
@@ -39,11 +39,12 @@ def make_link(source, target):
sys.stdout.flush()
os.rmdir(target)
+ print 'Creating link from %s to %s' % (source, target)
+ sys.stdout.flush()
+
if utils.GuessOS() == 'win32':
- source = os.path.relpath(source)
return subprocess.call(['mklink', '/j', target, source], shell=True)
else:
- source = os.path.relpath(source, start=target)
return subprocess.call(['ln', '-s', source, target])
@@ -56,6 +57,10 @@ def main(argv):
name = source
# Remove any addtional path components preceding TARGET_NAME.
(path, name) = os.path.split(name)
+ if utils.GuessOS() == 'win32':
+ source = os.path.relpath(source)
+ else:
+ source = os.path.relpath(source, start=target)
exit_code = make_link(source, os.path.join(target, name))
if exit_code != 0:
return exit_code
« no previous file with comments | « no previous file | dart/tools/testing/dart/test_suite.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698