Fix test/mac/gyptest-strip.py with Xcode 5.0 or higher

Fix test/mac/gyptest-strip.py with Xcode 5.0 or higher

The version of clang that ships with Xcode 5.0 and higher add an extra
symbol if the command-line use a relative path with more than one
component to reference the source file (like ./file.c). This extra
symbol is not present if using an absolute path to the file, and cause
a difference of behaviour between xcodebuild and ninja.

Account for the extra symbol in the un-stripped binaries when building
with ninja and using Xcode 5.0 or higher. A possible alternative would
be to always use absolute path with ninja, but it would make the build
environment non-relocatable.

BUG=gyp:381
R=mark@chromium.org

Committed: https://code.google.com/p/gyp/source/detail?r=1869

[sdefresne, 2014-03-12 13:00:43]

PTAL

[Mark Mentovai, 2014-03-12 13:29:19]

LGTM

I found that it’s not really relative paths that are the problem, a path in the
current directory like “test.cc” is treated the same as an absolute path like
“/Users/mark/test.cc”. The difference happens with relative paths with more than
one component, like “./test.cc”.

https://codereview.chromium.org/197073002/diff/1/test/mac/gyptest-strip.py
File test/mac/gyptest-strip.py (right):

https://codereview.chromium.org/197073002/diff/1/test/mac/gyptest-strip.py#ne...
test/mac/gyptest-strip.py:39: # Starting with Xcode 5.0, clang add an additional
symbols to the compiled
adds an additional symbol

https://codereview.chromium.org/197073002/diff/1/test/mac/gyptest-strip.py#ne...
test/mac/gyptest-strip.py:41: # with Xcode 5.0 or higher, take this additional
symbol in consideration
into consideration

https://codereview.chromium.org/197073002/diff/1/test/mac/gyptest-strip.py#ne...
test/mac/gyptest-strip.py:42: # for no stripped builds (it is stripped by all
strip commands).
unstripped builds

[sdefresne, 2014-03-12 13:42:18]

Thank you for the reviews.

https://codereview.chromium.org/197073002/diff/1/test/mac/gyptest-strip.py
File test/mac/gyptest-strip.py (right):

https://codereview.chromium.org/197073002/diff/1/test/mac/gyptest-strip.py#ne...
test/mac/gyptest-strip.py:39: # Starting with Xcode 5.0, clang add an additional
symbols to the compiled
On 2014/03/12 13:29:19, Mark Mentovai wrote:
> adds an additional symbol

Done.

https://codereview.chromium.org/197073002/diff/1/test/mac/gyptest-strip.py#ne...
test/mac/gyptest-strip.py:41: # with Xcode 5.0 or higher, take this additional
symbol in consideration
On 2014/03/12 13:29:19, Mark Mentovai wrote:
> into consideration

Done.

https://codereview.chromium.org/197073002/diff/1/test/mac/gyptest-strip.py#ne...
test/mac/gyptest-strip.py:42: # for no stripped builds (it is stripped by all
strip commands).
On 2014/03/12 13:29:19, Mark Mentovai wrote:
> unstripped builds

Done.

[sdefresne, 2014-03-12 13:47:33]

Committed patchset #2 manually as r1869 (presubmit successful).