Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(834)

Unified Diff: test/mac/gyptest-strip.py

Issue 23600042: ninja&make/mac: Only pass -x for loadable_modules. (Closed) Base URL: http://gyp.googlecode.com/svn/trunk/
Patch Set: Created 7 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: test/mac/gyptest-strip.py
===================================================================
--- test/mac/gyptest-strip.py (revision 1721)
+++ test/mac/gyptest-strip.py (working copy)
@@ -39,15 +39,15 @@
# The actual numbers here are not interesting, they just need to be the same
# in both the xcode and the make build.
- CheckNsyms(OutPath('no_postprocess'), 10)
- CheckNsyms(OutPath('no_strip'), 10)
+ CheckNsyms(OutPath('no_postprocess'), 29)
+ CheckNsyms(OutPath('no_strip'), 29)
CheckNsyms(OutPath('strip_all'), 0)
- CheckNsyms(OutPath('strip_nonglobal'), 2)
- CheckNsyms(OutPath('strip_debugging'), 2)
+ CheckNsyms(OutPath('strip_nonglobal'), 6)
+ CheckNsyms(OutPath('strip_debugging'), 7)
CheckNsyms(OutPath('strip_all_custom_flags'), 0)
CheckNsyms(test.built_file_path(
'strip_all_bundle.framework/Versions/A/strip_all_bundle', chdir='strip'),
0)
- CheckNsyms(OutPath('strip_save'), 2)
+ CheckNsyms(OutPath('strip_save'), 7)
test.pass_test()

Powered by Google App Engine
This is Rietveld 408576698