.gyp file for Breakpad on OS X.

.gyp file for Breakpad on OS X.

[jeremy, 2009-04-20 23:40:12]

Please disregard DEPS changes.

This requires http://codereview.chromium.org/87013 to work + rolling breakpad to
ToT.

Mark: Is there a way I can specify that anything that includes breakpadUtilities
needs to link against libcrypto?

[John Grabowski, 2009-04-20 23:48:55]

LGTM but wait for mark

http://codereview.chromium.org/87014/diff/1/3
File breakpad/breakpad.gyp (right):

http://codereview.chromium.org/87014/diff/1/3#newcode1
Line 1: # Copyright (c) 2009 The Chromium Authors. All rights reserved.
Be clear that this is only implemented for the Mac (at the top of this file)

http://codereview.chromium.org/87014/diff/1/4
File chrome/chrome.gyp (right):

http://codereview.chromium.org/87014/diff/1/4#newcode1852
Line 1852: '../third_party/icu38/icu38.gyp:icui18n',
What is this and why is it needed as part of the gyp change?

[Mark Mentovai, 2009-04-21 00:42:12]

Mostly lg

http://codereview.chromium.org/87014/diff/1004/9
File breakpad/breakpad.gyp (right):

http://codereview.chromium.org/87014/diff/1004/9#newcode18
Line 18: 'target_name': 'breakpadUtilities',
This would be a first for interCapping within our target names.  Please use
breakpad_utilities instead.  Obviously you'll also need to change the
"dependencies" sections too.

http://codereview.chromium.org/87014/diff/1004/9#newcode46
Line 46: 'OTHER_LDFLAGS': ['-lcrypto',],
Use this instead:

'link_settings': {'libraries': ['$(SDKROOT)/usr/lib/libcrypto.dylib']}

http://codereview.chromium.org/87014/diff/1004/9#newcode48
Line 48: 'libraries': [
'libraries' should appear within a 'link_settings' section. 
http://code.google.com/p/gyp/wiki/InputFormatReference has some information on
link_settings.

http://codereview.chromium.org/87014/diff/1004/9#newcode72
Line 72: 'OTHER_LDFLAGS': ['-lcrypto',],
Again, remove.

http://codereview.chromium.org/87014/diff/1004/9#newcode74
Line 74: 'libraries': [
Put in link_settings as above.

http://codereview.chromium.org/87014/diff/1004/11
File chrome/chrome.gyp (right):

http://codereview.chromium.org/87014/diff/1004/11#newcode1661
Line 1661: # Needed for breakpad.
Nope, take this out.  If you get link_settings right, this isn't needed. 
Dependency targets (like breakpad in this case) should carry their own
information about libs they need, you shouldn't have to put stuff like this all
over the place just because you've added a dependency.