Make GN public configs apply libs to targets.

Make GN public configs apply libs to targets.

Previously, putting libs or lib_dirs in a config that a target exports as a public config would not result in those values getting applied to dependant targets.

This patch applies those to the targets how one would expect by resolving the public configs before the libs are copied. This involved splitting up pulling dependencies' information into a configs phase (which must happen before we update the libs on the target) and the libs phase (which must happen after so the dependencies' libs come after).

Clarify ordering in the help.

Committed: https://crrev.com/567cb68891e32e2a398302b2b2405e4250ccc1d7
Cr-Commit-Position: refs/heads/master@{#364749}

[brettw, 2015-12-11 00:33:40]

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

[brettw, 2015-12-11 00:33:47]

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

[commit-bot: I haz the power, 2015-12-11 00:37:04]

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

[commit-bot: I haz the power, 2015-12-11 00:50:06]

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

[commit-bot: I haz the power, 2015-12-11 00:50:07]

Dry run: Try jobs failed on following builders:
  linux_chromium_compile_dbg_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[brettw, 2015-12-11 03:32:25]

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

[commit-bot: I haz the power, 2015-12-11 03:33:54]

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

[commit-bot: I haz the power, 2015-12-11 03:48:53]

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

[commit-bot: I haz the power, 2015-12-11 03:48:54]

Dry run: This issue passed the CQ dry run.

[Patrick Monette, 2015-12-11 17:31:04]

lgtm

[Patrick Monette, 2015-12-11 17:31:06]

lgtm

lgtm

[Patrick Monette, 2015-12-11 17:31:06]

lgtm

lgtm

lgtm

[Patrick Monette, 2015-12-11 17:31:06]

lgtm

lgtm

lgtm

lgtm

[Patrick Monette, 2015-12-11 17:31:06]

lgtm

lgtm

lgtm

lgtm

lgtm

[Patrick Monette, 2015-12-11 17:31:09]

lgtm

lgtm

lgtm

lgtm

lgtm

lgtm

[Patrick Monette, 2015-12-11 17:31:13]

lgtm

[Patrick Monette, 2015-12-11 17:31:13]

lgtm

[Patrick Monette, 2015-12-11 17:31:46]

On 2015/12/11 17:31:13, Patrick Monette wrote:
> lgtm

Sorry I just learned what quick lgtm does!

[brettw, 2015-12-11 18:17:02]

The CQ bit was checked by brettw@chromium.org

[commit-bot: I haz the power, 2015-12-11 18:19:23]

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

[commit-bot: I haz the power, 2015-12-11 18:38:35]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-12-11 18:40:23]

Description was changed from

==========
Make GN public configs apply libs to targets.

Previously, putting libs or lib_dirs in a config that a target exports as a
public config would not result in those values getting applied to dependant
targets.

This patch applies those to the targets how one would expect by resolving the
public configs before the libs are copied. This involved splitting up pulling
dependencies' information into a configs phase (which must happen before we
update the libs on the target) and the libs phase (which must happen after so
the dependencies' libs come after).

Clarify ordering in the help.
==========

to

==========
Make GN public configs apply libs to targets.

Previously, putting libs or lib_dirs in a config that a target exports as a
public config would not result in those values getting applied to dependant
targets.

This patch applies those to the targets how one would expect by resolving the
public configs before the libs are copied. This involved splitting up pulling
dependencies' information into a configs phase (which must happen before we
update the libs on the target) and the libs phase (which must happen after so
the dependencies' libs come after).

Clarify ordering in the help.

Committed: https://crrev.com/567cb68891e32e2a398302b2b2405e4250ccc1d7
Cr-Commit-Position: refs/heads/master@{#364749}
==========

[commit-bot: I haz the power, 2015-12-11 18:40:24]

Patchset 1 (id:??) landed as
https://crrev.com/567cb68891e32e2a398302b2b2405e4250ccc1d7
Cr-Commit-Position: refs/heads/master@{#364749}