GN: Don't write ldflags and libs when unneeded.

GN: Don't write ldflags and libs when unneeded.

Previously ldflags and libs were emitted for every linked target including static
libraries. These variables are documented to not apply to static libraries and
are unneeded in that case.

Improve the documentation for "libs".

BUG=570015
Committed: https://crrev.com/b2c920d22ae5721ce4a32a8dc524d13b08356593
Cr-Commit-Position: refs/heads/master@{#373111}

[brettw, 2016-02-02 23:46:55]

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

[brettw, 2016-02-02 23:46:59]

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

[commit-bot: I haz the power, 2016-02-02 23:52:43]

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

[brettw, 2016-02-02 23:53:40]

Description was changed from

==========
GN: Don't write ldflags and libs when unneeded.

Previously ldflags and libs were emitted for every linked target include static
libraries. These variables are documented to not apply to static libraries and
are unneeded in that case.

Improve the documentation for "libs".

BUG=570015
==========

to

==========
GN: Don't write ldflags and libs when unneeded.

Previously ldflags and libs were emitted for every linked target including
static
libraries. These variables are documented to not apply to static libraries and
are unneeded in that case.

Improve the documentation for "libs".

BUG=570015
==========

[Dirk Pranke, 2016-02-02 23:55:31]

lgtm, but can you update //tools/gn/docs/reference.md as well?

[brettw, 2016-02-03 00:23:42]

On 2016/02/02 23:55:31, Dirk Pranke wrote:
> lgtm, but can you update //tools/gn/docs/reference.md as well?

Done

[brettw, 2016-02-03 00:23:44]

The CQ bit was checked by brettw@chromium.org

[brettw, 2016-02-03 00:23:45]

The patchset sent to the CQ was uploaded after l-g-t-m from dpranke@chromium.org
Link to the patchset: https://codereview.chromium.org/1660213002/#ps20001
(title: "Update reference")

[commit-bot: I haz the power, 2016-02-03 00:26:34]

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

[commit-bot: I haz the power, 2016-02-03 01:18:34]

Description was changed from

==========
GN: Don't write ldflags and libs when unneeded.

Previously ldflags and libs were emitted for every linked target including
static
libraries. These variables are documented to not apply to static libraries and
are unneeded in that case.

Improve the documentation for "libs".

BUG=570015
==========

to

==========
GN: Don't write ldflags and libs when unneeded.

Previously ldflags and libs were emitted for every linked target including
static
libraries. These variables are documented to not apply to static libraries and
are unneeded in that case.

Improve the documentation for "libs".

BUG=570015
==========

[commit-bot: I haz the power, 2016-02-03 01:18:34]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-02-03 01:19:53]

Description was changed from

==========
GN: Don't write ldflags and libs when unneeded.

Previously ldflags and libs were emitted for every linked target including
static
libraries. These variables are documented to not apply to static libraries and
are unneeded in that case.

Improve the documentation for "libs".

BUG=570015
==========

to

==========
GN: Don't write ldflags and libs when unneeded.

Previously ldflags and libs were emitted for every linked target including
static
libraries. These variables are documented to not apply to static libraries and
are unneeded in that case.

Improve the documentation for "libs".

BUG=570015

Committed: https://crrev.com/b2c920d22ae5721ce4a32a8dc524d13b08356593
Cr-Commit-Position: refs/heads/master@{#373111}
==========

[commit-bot: I haz the power, 2016-02-03 01:19:54]

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