Fixing make generator handling of multiple actions in a dependent target.

Fixing make generator handling of multiple actions in a dependent target.

Multiple actions in a single none target fail to all depend correctly on
the stamps from all the targets that they are in depend on.

test/actions-multiple has been enhanced to cover this case.

BUG=https://code.google.com/p/chromium/issues/detail?id=129814
TEST=test/actions-multiple
R=thakis@chromium.org
Committed: https://code.google.com/p/gyp/source/detail?r=1403

[Nico, 2012-05-28 16:43:53]

http://codereview.chromium.org/10454038/diff/1/test/actions-multiple/gyptest-...
File test/actions-multiple/gyptest-all.py (right):

http://codereview.chromium.org/10454038/diff/1/test/actions-multiple/gyptest-...
test/actions-multiple/gyptest-all.py:22: # This is only relevant to the make
generator.
Why can't you run this test for other generators?

[bradn, 2012-05-29 21:21:47]

PTAL

http://codereview.chromium.org/10454038/diff/1/test/actions-multiple/gyptest-...
File test/actions-multiple/gyptest-all.py (right):

http://codereview.chromium.org/10454038/diff/1/test/actions-multiple/gyptest-...
test/actions-multiple/gyptest-all.py:22: # This is only relevant to the make
generator.
On 2012/05/28 16:43:53, Nico wrote:
> Why can't you run this test for other generators?

You can only do this with generators / build tools that can be asked to build
just a single file and its transitive dependencies (make and ninja).
I've altered the CL to include ninja too.

[Nico, 2012-05-29 21:33:08]

I think this is fine.

http://codereview.chromium.org/10454038/diff/6/test/actions-multiple/gyptest-...
File test/actions-multiple/gyptest-all.py (right):

http://codereview.chromium.org/10454038/diff/6/test/actions-multiple/gyptest-...
test/actions-multiple/gyptest-all.py:22: # depend on another 'none' target cause
depended on target to
this doesn't parse for me

http://codereview.chromium.org/10454038/diff/6/test/actions-multiple/gyptest-...
test/actions-multiple/gyptest-all.py:29: target='multi2.txt'
spaces around =

[bradn, 2012-05-29 21:44:26]

PTAL

http://codereview.chromium.org/10454038/diff/6/test/actions-multiple/gyptest-...
File test/actions-multiple/gyptest-all.py (right):

http://codereview.chromium.org/10454038/diff/6/test/actions-multiple/gyptest-...
test/actions-multiple/gyptest-all.py:22: # depend on another 'none' target cause
depended on target to
On 2012/05/29 21:33:08, Nico wrote:
> this doesn't parse for me

Done.

http://codereview.chromium.org/10454038/diff/6/test/actions-multiple/gyptest-...
test/actions-multiple/gyptest-all.py:29: target='multi2.txt'
On 2012/05/29 21:33:08, Nico wrote:
> spaces around =

Done.

[Nico, 2012-05-29 21:48:06]

lgtm