Moves warning about invalid targets into 'invalid_targets' in result

Moves warning about invalid targets into 'invalid_targets' in result

Previously if an invalid_target was supplied analyze would report it
via 'warning'. I'm moving that warning to 'invalid_targets' and
providing the list of invalid targets there.

BUG=426892
TEST=covered by tests
R=scottmg@chromium.org

Committed: https://code.google.com/p/gyp/source/detail?r=1994

[sky, 2014-10-24 16:18:38]

sky@chromium.org changed reviewers:
+ scottmg@chromium.org

[scottmg, 2014-10-24 16:27:58]

lgtm

https://codereview.chromium.org/671233004/diff/20001/pylib/gyp/generator/anal...
File pylib/gyp/generator/analyzer.py (right):

https://codereview.chromium.org/671233004/diff/20001/pylib/gyp/generator/anal...
pylib/gyp/generator/analyzer.py:14: warning: only supplied if there is a
warning.
delete this

[sky, 2014-10-24 16:30:12]

https://codereview.chromium.org/671233004/diff/20001/pylib/gyp/generator/anal...
File pylib/gyp/generator/analyzer.py (right):

https://codereview.chromium.org/671233004/diff/20001/pylib/gyp/generator/anal...
pylib/gyp/generator/analyzer.py:14: warning: only supplied if there is a
warning.
On 2014/10/24 16:27:58, scottmg wrote:
> delete this

Done.

[sky, 2014-10-24 16:30:17]

The CQ bit was checked by sky@chromium.org

[commit-bot: I haz the power, 2014-10-24 16:31:36]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-10-24 16:31:37]

Commit queue rejected this change because it did not recognize the base URL.
Please commit your change manually.

[sky, 2014-10-24 18:16:18]

Committed patchset #3 (id:40001) manually as r1994 (presubmit successful).