Add a script to fetch the latest SKPs.

Add a script to fetch the latest SKPs.

I keep forgetting how best to do this.

NOTRY=true
NOTREECHECKS=true

Committed: https://skia.googlesource.com/skia/+/674cd7e05c79ee60216c4d0fd53c2c0e162a7887

[mtklein, 2015-01-28 15:31:25]

mtklein@google.com changed reviewers:
+ halcanary@google.com, mtklein@google.com

[hal.canary, 2015-01-28 15:37:02]

https://codereview.chromium.org/880283002/diff/1/bin/fetch-skps
File bin/fetch-skps (right):

https://codereview.chromium.org/880283002/diff/1/bin/fetch-skps#newcode3
bin/fetch-skps:3: gsutil -m cp -r gs://chromium-skia-gm/playback_`cat
SKP_VERSION`/skps .
#!/bin/sh

version="$(cat "$(cd "$(dirname "$0")"; pwd)/../SKP_VERSION")"
gsutil -m cp -r gs://chromium-skia-gm/playback_${version}/skps .

[mtklein, 2015-01-28 15:45:58]

> version="$(cat "$(cd "$(dirname "$0")"; pwd)/../SKP_VERSION")"
> gsutil -m cp -r gs://chromium-skia-gm/playback_${version}/skps .

ptal

[hal.canary, 2015-01-28 15:48:44]

On 2015/01/28 15:45:58, mtklein wrote:
> > version="$(cat "$(cd "$(dirname "$0")"; pwd)/../SKP_VERSION")"
> > gsutil -m cp -r gs://chromium-skia-gm/playback_${version}/skps .
> 
> ptal

LGTM

If you are crazy enough to store your skia checkout inside a directory with
spaces, and then call this script from outside that directory, you deserve what
you get.

[hal.canary, 2015-01-28 15:49:03]

The CQ bit was checked by halcanary@google.com

[mtklein, 2015-01-28 15:49:33]

On 2015/01/28 15:48:44, Hal Canary wrote:
> On 2015/01/28 15:45:58, mtklein wrote:
> > > version="$(cat "$(cd "$(dirname "$0")"; pwd)/../SKP_VERSION")"
> > > gsutil -m cp -r gs://chromium-skia-gm/playback_${version}/skps .
> > 
> > ptal
> 
> LGTM
> 
> If you are crazy enough to store your skia checkout inside a directory with
> spaces, and then call this script from outside that directory, you deserve
what
> you get.

Yes.  And you can probably read and edit a shell script...

[commit-bot: I haz the power, 2015-01-28 15:49:33]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/880283002/40001

[hal.canary, 2015-01-28 15:51:35]

On 2015/01/28 15:49:33, mtklein wrote:
> On 2015/01/28 15:48:44, Hal Canary wrote:
> > If you are crazy enough to store your skia checkout inside a directory with
> > spaces, and then call this script from outside that directory, you deserve
> > what you get.
> 
> Yes.  And you can probably read and edit a shell script...

The set of people who put spaces in file names and the set of people who can
diagnose errors in Posix shell scripts are mutually exclusive.

[mtklein, 2015-01-28 16:04:10]

> The set of people who put spaces in file names and the set of people who can
> diagnose errors in Posix shell scripts are mutually exclusive.

In that case, we strongly don't care about them or expect them to fetch SKPs.

[mtklein, 2015-01-28 17:38:31]

The CQ bit was unchecked by mtklein@google.com

[mtklein, 2015-01-28 17:39:01]

The CQ bit was checked by mtklein@google.com

[commit-bot: I haz the power, 2015-01-28 17:39:04]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/880283002/40001

[commit-bot: I haz the power, 2015-01-28 17:39:13]

Committed patchset #3 (id:40001) as
https://skia.googlesource.com/skia/+/674cd7e05c79ee60216c4d0fd53c2c0e162a7887