bin/sync-and-gyp: gclient hackery

bin/sync-and-gyp: gclient hackery

Committed: https://skia.googlesource.com/skia/+/1f17364dc86de6f3e537440cd67cdb952b5e39f8

[hal.canary, 2015-10-30 22:06:53]

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

[hal.canary, 2015-10-30 22:06:53]

ptal

[mtklein, 2015-11-01 22:20:40]

lgtm

https://codereview.chromium.org/1427183003/diff/20001/bin/sync-and-gyp
File bin/sync-and-gyp (right):

https://codereview.chromium.org/1427183003/diff/20001/bin/sync-and-gyp#newcode28
bin/sync-and-gyp:28: printf ',s/"skia"/"."/\nwq\n' | ed .gclient
Why not sed?

[hal.canary, 2015-11-02 18:11:25]

On 2015/11/01 22:20:40, mtklein wrote:
> lgtm
> 
> https://codereview.chromium.org/1427183003/diff/20001/bin/sync-and-gyp
> File bin/sync-and-gyp (right):
> 
>
https://codereview.chromium.org/1427183003/diff/20001/bin/sync-and-gyp#newcode28
> bin/sync-and-gyp:28: printf ',s/"skia"/"."/\nwq\n' | ed .gclient
> Why not sed?

sed'd in-place editing has different syntax for BSD and GNU, it is not part of
POSIX.  So I learned something new, which is designed for in-place editing.

[hal.canary, 2015-11-02 18:11:37]

The CQ bit was checked by halcanary@google.com

[commit-bot: I haz the power, 2015-11-02 18:11:41]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1427183003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1427183003/20001

[commit-bot: I haz the power, 2015-11-02 18:22:46]

Committed patchset #2 (id:20001) as
https://skia.googlesource.com/skia/+/1f17364dc86de6f3e537440cd67cdb952b5e39f8