Automatically update submodule config entries on sync.

Automatically update submodule config entries on sync.

The effect of this is to preserve the behavior of `git status`,
i.e., only changes in the top-level repository will be reported.

Updated test expectations; I have no idea what I'm doing.

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=140396

[M-A Ruel, 2012-05-31 01:35:51]

https://chromiumcodereview.appspot.com/10454088/diff/1/tests/gclient_smoketes...
File tests/gclient_smoketest.py (right):

https://chromiumcodereview.appspot.com/10454088/diff/1/tests/gclient_smoketes...
tests/gclient_smoketest.py:142: self.assertEquals(len(results), len(items),
(stdout, items, len(items), len(results)))
please wrap

https://chromiumcodereview.appspot.com/10454088/diff/1/tests/gclient_smoketes...
tests/gclient_smoketest.py:800: ['running', ('running', self.root_dir + '/src'),
BTW, you should asked for help here, I guess you struggled a bit with it.

https://chromiumcodereview.appspot.com/10454088/diff/3001/gclient_scm.py
File gclient_scm.py (right):

https://chromiumcodereview.appspot.com/10454088/diff/3001/gclient_scm.py#newc...
gclient_scm.py:199: 'submodule.$name.ignore', '||',
'||' probably doesn't work on windows.

[szager1, 2012-05-31 06:39:44]

https://chromiumcodereview.appspot.com/10454088/diff/1/tests/gclient_smoketes...
File tests/gclient_smoketest.py (right):

https://chromiumcodereview.appspot.com/10454088/diff/1/tests/gclient_smoketes...
tests/gclient_smoketest.py:142: self.assertEquals(len(results), len(items),
(stdout, items, len(items), len(results)))
On 2012/05/31 01:35:52, Marc-Antoine Ruel wrote:
> please wrap

Done.

https://chromiumcodereview.appspot.com/10454088/diff/1/tests/gclient_smoketes...
tests/gclient_smoketest.py:800: ['running', ('running', self.root_dir + '/src'),
On 2012/05/31 01:35:52, Marc-Antoine Ruel wrote:
> BTW, you should asked for help here, I guess you struggled a bit with it.

I must say, these expectations were really laborious to update, and it's not at
all clear to me what exactly is being tested, other than gclient not completely
self-destructing.

https://chromiumcodereview.appspot.com/10454088/diff/3001/gclient_scm.py
File gclient_scm.py (right):

https://chromiumcodereview.appspot.com/10454088/diff/3001/gclient_scm.py#newc...
gclient_scm.py:199: 'submodule.$name.ignore', '||',
On 2012/05/31 01:35:52, Marc-Antoine Ruel wrote:
> '||' probably doesn't work on windows.

I tried it.  Works for me.  git-submodule is a shell script that runs under msys
bash.  So, all subprocesses of git-submodule also run in msys bash.

[cmp, 2012-05-31 20:43:06]

lgtm, but you should wait for maruel's or nsylvain's lgtm, too :)

[nsylvain, 2012-06-01 01:29:09]

I'm OK with the change although I dont think I know this code enough to make
sure you did not forget any other place to put your new call.

[M-A Ruel, 2012-06-01 01:57:50]

lgtm

https://chromiumcodereview.appspot.com/10454088/diff/1/tests/gclient_smoketes...
File tests/gclient_smoketest.py (right):

https://chromiumcodereview.appspot.com/10454088/diff/1/tests/gclient_smoketes...
tests/gclient_smoketest.py:800: ['running', ('running', self.root_dir + '/src'),
On 2012/05/31 06:39:45, szager1 wrote:
> On 2012/05/31 01:35:52, Marc-Antoine Ruel wrote:
> > BTW, you should asked for help here, I guess you struggled a bit with it.
> 
> I must say, these expectations were really laborious to update, and it's not
at
> all clear to me what exactly is being tested, other than gclient not
completely
> self-destructing.

Outputs used to be flaky. This assert that all the headers were printed
correctly.

[szager1, 2012-06-04 22:06:07]

FYI: playing with this a bit, I saw that the `git submodule` calls produce way
too much noise for simple gclient operations.  Because this is non-essential
functionality, I'm suppressing the output.  The test expectations don't need to
change.

[commit-bot: I haz the power, 2012-06-04 22:06:26]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/szager@chromium.org/10454088/13001

[commit-bot: I haz the power, 2012-06-04 22:13:57]

Change committed as 140396