Apply arflags to dependent configs in GN.

Apply arflags to dependent configs in GN.

Previously they were not copied from child configs so they wouldn't work in
all cases.

This also adds arflags support for "gn desc", and changes ldflags to no longer be printed for static libraries and source sets (since it doesn't apply, this is unnecessary and confusing).

Committed: https://crrev.com/10b4460a4180b1cff25517c1e4cdae3b9e5c02cb
Cr-Commit-Position: refs/heads/master@{#389565}

[brettw, 2016-04-25 18:35:11]

Description was changed from

==========
Apply arflags to dependent configs in GN.

Previously they were not copied from child configs so they wouldn't work in
all cases.

This also adds arflags support for "gn desc".
==========

to

==========
Apply arflags to dependent configs in GN.

Previously they were not copied from child configs so they wouldn't work in
all cases.

This also adds arflags support for "gn desc", and changes ldflags to no longer
be printed for static libraries and source sets (since it doesn't apply, this is
unnecessary and confusing).
==========

[brettw, 2016-04-25 18:35:59]

brettw@chromium.org changed reviewers:
+ brucedawson@chromium.org

[brettw, 2016-04-25 18:36:00]

I don't have a brilliant idea about how to verify that you don't forget to do
this for new flags. I think the new warnings will work better, though.

[brettw, 2016-04-25 18:36:03]

The CQ bit was checked by brettw@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-04-25 18:36:21]

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

[brucedawson, 2016-04-25 19:52:52]

lgtm

[commit-bot: I haz the power, 2016-04-25 20:15:20]

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

[commit-bot: I haz the power, 2016-04-25 20:15:21]

Dry run: This issue passed the CQ dry run.

[brettw, 2016-04-25 21:23:32]

The CQ bit was checked by brettw@chromium.org

[commit-bot: I haz the power, 2016-04-25 21:24:15]

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

[commit-bot: I haz the power, 2016-04-25 21:30:50]

Description was changed from

==========
Apply arflags to dependent configs in GN.

Previously they were not copied from child configs so they wouldn't work in
all cases.

This also adds arflags support for "gn desc", and changes ldflags to no longer
be printed for static libraries and source sets (since it doesn't apply, this is
unnecessary and confusing).
==========

to

==========
Apply arflags to dependent configs in GN.

Previously they were not copied from child configs so they wouldn't work in
all cases.

This also adds arflags support for "gn desc", and changes ldflags to no longer
be printed for static libraries and source sets (since it doesn't apply, this is
unnecessary and confusing).
==========

[commit-bot: I haz the power, 2016-04-25 21:30:51]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-25 21:32:40]

Description was changed from

==========
Apply arflags to dependent configs in GN.

Previously they were not copied from child configs so they wouldn't work in
all cases.

This also adds arflags support for "gn desc", and changes ldflags to no longer
be printed for static libraries and source sets (since it doesn't apply, this is
unnecessary and confusing).
==========

to

==========
Apply arflags to dependent configs in GN.

Previously they were not copied from child configs so they wouldn't work in
all cases.

This also adds arflags support for "gn desc", and changes ldflags to no longer
be printed for static libraries and source sets (since it doesn't apply, this is
unnecessary and confusing).

Committed: https://crrev.com/10b4460a4180b1cff25517c1e4cdae3b9e5c02cb
Cr-Commit-Position: refs/heads/master@{#389565}
==========

[commit-bot: I haz the power, 2016-04-25 21:32:41]

Patchset 2 (id:??) landed as
https://crrev.com/10b4460a4180b1cff25517c1e4cdae3b9e5c02cb
Cr-Commit-Position: refs/heads/master@{#389565}