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

Unified Diff: build/common.gypi

Issue 113652: Added a script to remove headers from the target.... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 11 years, 7 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
« no previous file with comments | « no previous file | build/mac/remove_target_headers » ('j') | build/mac/remove_target_headers » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/common.gypi
===================================================================
--- build/common.gypi (revision 16417)
+++ build/common.gypi (working copy)
@@ -348,6 +348,20 @@
['_mac_bundle', {
'xcode_settings': {'OTHER_LDFLAGS': ['-Wl,-ObjC']},
}],
+ ['_type=="executable" and _mac_bundle', {
+ 'postbuilds': [
+ {
+ 'variables': {
+ # Define remove_target_headers in a variable ending in _path
+ # so that gyp understands it's a path and performs proper
+ # relativization during dict merging.
+ 'remove_target_headers_path': 'mac/remove_target_headers',
+ },
+ 'postbuild_name': 'Remove Target Headers',
+ 'action': ['<(remove_target_headers_path)'],
+ },
+ ],
+ }],
['_type=="executable"', {
'postbuilds': [
{
« no previous file with comments | « no previous file | build/mac/remove_target_headers » ('j') | build/mac/remove_target_headers » ('J')

Powered by Google App Engine
This is Rietveld 408576698