Browse Source

deps: cherry-pick 1383d00 from v8 upstream

Original commit message:

    tools: fix tickprocessor Cpp symbols on mac

    Despite man page documentation:

        -f Display the symbol table of a dynamic library flat (as one
           file not separate modules).

    `nm` on mac treats `-f` as a shorthand for `-format`. The `-f` argument
    does not seem to be required, so just remove it completely.

    (For `-format` documentation - see `nm --help` on mac).

    BUG=

    Review URL: https://codereview.chromium.org/1840633002

    Cr-Commit-Position: refs/heads/master@{#35445}

Fix: #5903
PR-URL: https://github.com/nodejs/node/pull/6179
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Michaël Zasso <mic.besace@gmail.com>
process-exit-stdio-flushing
Fedor Indutny 9 years ago
parent
commit
a11d506dec
  1. 5
      deps/v8/tools/tickprocessor.js

5
deps/v8/tools/tickprocessor.js

@ -757,8 +757,11 @@ inherits(MacCppEntriesProvider, UnixCppEntriesProvider);
MacCppEntriesProvider.prototype.loadSymbols = function(libName) { MacCppEntriesProvider.prototype.loadSymbols = function(libName) {
this.parsePos = 0; this.parsePos = 0;
libName = this.targetRootFS + libName; libName = this.targetRootFS + libName;
// It seems that in OS X `nm` thinks that `-f` is a format option, not a
// "flat" display option flag.
try { try {
this.symbols = [os.system(this.nmExec, ['-n', '-f', libName], -1, -1), '']; this.symbols = [os.system(this.nmExec, ['-n', libName], -1, -1), ''];
} catch (e) { } catch (e) {
// If the library cannot be found on this system let's not panic. // If the library cannot be found on this system let's not panic.
this.symbols = ''; this.symbols = '';

Loading…
Cancel
Save