Skip to content

Commit

Permalink
build: unbreak -prof, disable PIE on OS X
Browse files Browse the repository at this point in the history
Commit 204f3a8 ("build: Bump MACOSX_DEPLOYMENT_TARGET to 10.7")
unwittingly turned on new ASLR features that make `-prof` unusable for
profiling C++ code, breaking `test/parallel/test-tick-processor.js` in
the process.  Build with `-Wl,-no_pie` for now.

Fixes: nodejs#5903
  • Loading branch information
bnoordhuis committed Apr 28, 2016
1 parent 153b796 commit cc0d71f
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion common.gypi
Original file line number Diff line number Diff line change
Expand Up @@ -325,7 +325,12 @@
},
'target_conditions': [
['_type!="static_library"', {
'xcode_settings': {'OTHER_LDFLAGS': ['-Wl,-search_paths_first']},
'xcode_settings': {
'OTHER_LDFLAGS': [
'-Wl,-no_pie',
'-Wl,-search_paths_first',
],
},
}],
],
'conditions': [
Expand Down

0 comments on commit cc0d71f

Please sign in to comment.