Convenience script for fetching dartium

Convenience script for fetching dartium

R=terry@google.com

Committed: https://code.google.com/p/dart/source/detail?r=38672

[vsm, 2014-07-18 12:52:19]

On 2014/07/18 12:47:47, vsm wrote:

I've been using this to easily checkout writable git-svn-based copies of
different dartium branches.  E.g.,

python fetch_dartium.py --branch=dartium_integration --path=~/git/dartium37

to checkout the latest.

[terry, 2014-07-18 13:33:03]

lgtm

Worked except for --path with ~.  Had to use fully qualified path see
suggestion.

https://codereview.chromium.org/400183002/diff/1/tools/dartium/fetch_dartium.py
File tools/dartium/fetch_dartium.py (right):

https://codereview.chromium.org/400183002/diff/1/tools/dartium/fetch_dartium....
tools/dartium/fetch_dartium.py:59: path = options.path
My path is '/usr/local/google/home/terry' couldn't use'--path=~/dartium-2062'
had to use '--path=/usr/local/google/home/terry/dartium-2062'

I think you might want:

path = os.path.expanduser(path) to expand ~

[rmacnak, 2014-07-18 18:27:16]

https://codereview.chromium.org/400183002/diff/1/tools/dartium/fetch_dartium.py
File tools/dartium/fetch_dartium.py (right):

https://codereview.chromium.org/400183002/diff/1/tools/dartium/fetch_dartium....
tools/dartium/fetch_dartium.py:106: blink_fetch = Run(['git', 'svn', 'clone',
'-rHEAD', blink_url, 'blink'])
These are shallow checkouts without the other branches. We should add a warning
that they are not suitable for doing merges.

[vsm, 2014-07-29 14:48:42]

https://chromiumcodereview.appspot.com/400183002/diff/1/tools/dartium/fetch_d...
File tools/dartium/fetch_dartium.py (right):

https://chromiumcodereview.appspot.com/400183002/diff/1/tools/dartium/fetch_d...
tools/dartium/fetch_dartium.py:59: path = options.path
On 2014/07/18 13:33:03, terry wrote:
> My path is '/usr/local/google/home/terry' couldn't use'--path=~/dartium-2062'
> had to use '--path=/usr/local/google/home/terry/dartium-2062'
> 
> I think you might want:
> 
> path = os.path.expanduser(path) to expand ~

Done.

https://chromiumcodereview.appspot.com/400183002/diff/1/tools/dartium/fetch_d...
tools/dartium/fetch_dartium.py:106: blink_fetch = Run(['git', 'svn', 'clone',
'-rHEAD', blink_url, 'blink'])
On 2014/07/18 18:27:16, rmacnak wrote:
> These are shallow checkouts without the other branches. We should add a
warning
> that they are not suitable for doing merges.

Comment added at the top.  I use this to get going quickly (e.g., need to debug
trunk on a mac), so shallow usually makes sense.

We could add a --deep or similar at some point.

[vsm, 2014-07-29 14:59:39]

Committed patchset #2 manually as r38672 (presubmit successful).