Clearer help in gclient-new-workdir.py

gclient-new-workdir.py

Index: gclient-new-workdir.py
diff --git a/gclient-new-workdir.py b/gclient-new-workdir.py
index 91ba35b9efbd224ee6995770e1ccc06be0b7cd5f..153cb9703ee0cf14076f4340500ac96b4f227d71 100755
--- a/gclient-new-workdir.py
+++ b/gclient-new-workdir.py
@@ -14,21 +14,31 @@ import sys
 
 
 def parse_options(argv):
-  assert not sys.platform.startswith("win")
+  if sys.platform == "win32":
+    print >> sys.stderr, ("This script cannot run on Windows because it uses"
+                          "symlinks.")

[iannucci, 2013/11/13 17:57:30]

As M-A pointed out, could convert all "" quoted strings to '' quoted strings?
(at least the ones you touched, if you don't want to change the whole file.

Also, I prefer

print >> ..., (
    "This .... symlinks.")

especially since the string could all fit on a line.

+    sys.exit(1)
 
   if len(argv) != 3:
-    print("usage: gclient-new-workdir.py <repository> <new_workdir>")
+    print("""usage: gclient-new-workdir.py <repository> <new_workdir>
+
+    Clone an existing gclient directory, taking care of all sub-repositories

[M-A Ruel, 2013/11/13 17:17:33]

You want the text to be +4 at output?

[iannucci, 2013/11/13 17:57:30]

I think you want to pass this string through
http://docs.python.org/2/library/textwrap.html#textwrap.dedent so that you don't
get awkward indentation.

+    Works similarly to 'git new-workdir'.
+
+    <repository> must be a absolute path
+    <new_workdir> must not exist
+    """)
     sys.exit(1)
 
-  repository = argv[1]
+  repository = os.path.abspath(argv[1])
   new_workdir = argv[2]
 
   if not os.path.exists(repository):
-    print("Repository does not exist: " + repository)
+    print >> sys.stderr, ("Repository does not exist: " + repository)
     sys.exit(1)
 
   if os.path.exists(new_workdir):
-    print("New workdir already exists: " + new_workdir)
+    print >> sys.stderr, ("New workdir already exists: " + new_workdir)

[iannucci, 2013/11/13 17:57:30]

What do you think about a print_err function do this?

     sys.exit(1)
 
   return repository, new_workdir
@@ -39,11 +49,11 @@ def main(argv):
 
   gclient = os.path.join(repository, ".gclient")
   if not os.path.exists(gclient):
-    print("No .gclient file: " + gclient)
+    print >> sys.stderr, ("No .gclient file: " + gclient)
 
   gclient_entries = os.path.join(repository, ".gclient_entries")
   if not os.path.exists(gclient_entries):
-    print("No .gclient_entries file: " + gclient_entries)
+    print >> sys.stderr, ("No .gclient_entries file: " + gclient_entries)
 
   os.mkdir(new_workdir)
   os.symlink(gclient, os.path.join(new_workdir, ".gclient"))