-
Notifications
You must be signed in to change notification settings - Fork 29.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Error while running coverage #19057
Comments
I can reproduce on High Sierra. Bisecting now |
It's failed on my mac too. Apple's libtool is not support When I try to fix the coverage bug, I just use the CentOS. @trivikr Can you try the glibtool? |
I can reproduce this on High Sierra. And I dont know if i should create a new issue, but i found that
|
@jvelezpo Looks like it's the same error, so new issue is not required. |
I am having the same issue when running
hello.o /Users/nigelhanlon/Source/node/node/out/Release/obj.target/node_lib/src/tls_wrap.o
error: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/libtool: unknown option character `-' in: --coverage
Usage: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/libtool -static [-] file [...] [-filelist listfile[,dirname]] [-arch_only arch] [-sacLT] [-no_warning_for_no_symbols]
Usage: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/libtool -dynamic [-] file [...] [-filelist listfile[,dirname]] [-arch_only arch] [-o output] [-install_name name] [-compatibility_version #] [-current_version #] [-seg1addr 0x#] [-segs_read_only_addr 0x#] [-segs_read_write_addr 0x#] [-seg_addr_table <filename>] [-seg_addr_table_filename <file_system_path>] [-all_load] [-noall_load]
make[1]: *** [/Users/nigelhanlon/Source/node/node/out/Release/libnode.a] Error 1
rm 7080b3e45e89e1cd86c4d606e0421ef4647f1b29.intermediate
make: *** [node] Error 2 |
Did this ever work? Using |
@boneskull I seem to remember a PR that made coverage work on OSX but I never checked it out myself. |
This did use to work. (I submitted the PR). I haven't been able to figure out why it no longer works though. I may be able to dig into it again in the next few days |
@evanlucas Were you able to make any progress here? I'm not sure how I'd figure it out short of just bisecting. If you don't have time to look into it yourself, I was wondering if you had any hints? 😉 |
@boneskull nah I haven't been able to make any progress on it. I'd bet it has to do with changes to something that gets pulled in during |
I think #19094 may have fixed this, at least in part. I'll double-check, but it seems like I'm not getting cxx coverage, but am getting JS coverage. |
either way, it's not just dying with some fatal error |
Here's my output from running I get JS coverage, but no cxx coverage. This is macOS 10.13.4. |
I was able to work around this issue, and successfully run coverage, by removing all instances of |
Just ran the command again, and the error still exists: Command run:
Output:
|
This comment has been minimized.
This comment has been minimized.
Recent libtool on macOS does not support the --coverage flag that was being passed through in the final linking stage. This change patches gyp-mac-tool to not pass the --coverage flag through to libtool. It is now possible to generate code coverage for src/ on macOS. Fixes: nodejs#19057
@trivikr Still happening? |
Reran the following commands on latest commit in master 14707b6
The error still appears:
|
Still happening here. Any work arounds? I tried @JMGordon workaround and it seems to work :)
|
@nodejs/gyp |
did #24520 fix it? |
I suppose I can just find out myself. |
There's been no further activity in well over a year. Closing. |
I tried to run coverage by following the instructions given in https://github.com/nodejs/node/blob/master/BUILDING.md#building-nodejs-1
The following commands were run:
Coverage failed with the following error:
Is anyone else facing the same issue?
The text was updated successfully, but these errors were encountered: