Silence libtool warning when building libcrnet_standalone.a.

Silence libtool warning when building libcrnet_standalone.a.

Port code from tools/gyp/pylib/gyp/mac_tool.py to silence libtool
warning to ios/build/packaging/link_dependencies.py.

Add dependency on ios/build/packaging/link_dependencies.py for
target that invoke it so that they are rebuilt when the script
is updated.

BUG=None

Committed: https://crrev.com/db4a8df70e3521a6d555723356bdae37f95d7ff6
Cr-Commit-Position: refs/heads/master@{#373784}

[sdefresne, 2016-02-04 17:35:14]

sdefresne@chromium.org changed reviewers:
+ droger@chromium.org

[sdefresne, 2016-02-04 17:35:15]

Please take a look.

[sdefresne, 2016-02-04 17:36:03]

Description was changed from

==========
Silence libtool warning when building gsalib_standalone.a.

Port code from tools/gyp/pylib/gyp/mac_tool.py to silence libtool
warning to ios/build/packaging/link_dependencies.py.

Add dependency on ios/build/packaging/link_dependencies.py for
target that invoke it so that they are rebuilt when the script
is updated.

BUG=None
==========

to

==========
Silence libtool warning when building libcrnet_standalone.a.

Port code from tools/gyp/pylib/gyp/mac_tool.py to silence libtool
warning to ios/build/packaging/link_dependencies.py.

Add dependency on ios/build/packaging/link_dependencies.py for
target that invoke it so that they are rebuilt when the script
is updated.

BUG=None
==========

[droger, 2016-02-05 09:08:50]

https://codereview.chromium.org/1670613002/diff/1/ios/build/packaging/link_de...
File ios/build/packaging/link_dependencies.py (right):

https://codereview.chromium.org/1670613002/diff/1/ios/build/packaging/link_de...
ios/build/packaging/link_dependencies.py:137: if not libtool_re.match(line) and
not libtool_re5.match(line):
Where is libtool_re5 defined? Do we need to import something?

https://codereview.chromium.org/1670613002/diff/1/ios/build/packaging/link_de...
ios/build/packaging/link_dependencies.py:138: print >>sys.stderr, line
Missing whitespace after >>

[sdefresne, 2016-02-05 09:58:57]

PTAL

https://codereview.chromium.org/1670613002/diff/1/ios/build/packaging/link_de...
File ios/build/packaging/link_dependencies.py (right):

https://codereview.chromium.org/1670613002/diff/1/ios/build/packaging/link_de...
ios/build/packaging/link_dependencies.py:137: if not libtool_re.match(line) and
not libtool_re5.match(line):
On 2016/02/05 at 09:08:50, droger wrote:
> Where is libtool_re5 defined? Do we need to import something?

Ooops. Removed.

https://codereview.chromium.org/1670613002/diff/1/ios/build/packaging/link_de...
ios/build/packaging/link_dependencies.py:138: print >>sys.stderr, line
On 2016/02/05 at 09:08:50, droger wrote:
> Missing whitespace after >>

No, this is the syntax to write to a file using print statement.

[droger, 2016-02-05 10:06:33]

lgtm

[sdefresne, 2016-02-05 10:09:23]

The CQ bit was checked by sdefresne@chromium.org

[commit-bot: I haz the power, 2016-02-05 10:10:28]

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

[commit-bot: I haz the power, 2016-02-05 11:07:56]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-02-05 11:08:50]

Description was changed from

==========
Silence libtool warning when building libcrnet_standalone.a.

Port code from tools/gyp/pylib/gyp/mac_tool.py to silence libtool
warning to ios/build/packaging/link_dependencies.py.

Add dependency on ios/build/packaging/link_dependencies.py for
target that invoke it so that they are rebuilt when the script
is updated.

BUG=None
==========

to

==========
Silence libtool warning when building libcrnet_standalone.a.

Port code from tools/gyp/pylib/gyp/mac_tool.py to silence libtool
warning to ios/build/packaging/link_dependencies.py.

Add dependency on ios/build/packaging/link_dependencies.py for
target that invoke it so that they are rebuilt when the script
is updated.

BUG=None

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

[commit-bot: I haz the power, 2016-02-05 11:08:51]

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