-
-
Notifications
You must be signed in to change notification settings - Fork 181
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
Cannot run cargo-tarpulin in a pyo3 project #1092
Comments
I am facing the same issue :| @noamteyssier did you find any workaround ? |
Just an FYI I am working on this, the issue is EDIT: This can be tested via the |
And it all passes and works in examples I've checked so I'm going to go ahead and merge 🎉 |
@xd009642 I was testing your fix. I got this error on cargo-tarpaulin version: 0.25.0 when I tried to run the command
I am running this tool over piranha |
Did you add |
Ok. I tried that. I got :
...
|
So @ketkarameya I know this can be the worst thing to hear but "it works on my machine". At the very bottom is the output, also here is my
It'll be worth checking that to make sure there's nothing fishy going on
|
Thanks a lot. I will compare and check . I ll let you know. |
Actually, you know what, I ll try to get it running on CI (It's alright if I am unable to run it locally :) ) |
@xd009642 I have been trying to set up tarpaulin with github actions using the documentation. |
It looks like the LLVM version in the rust docker images doesn't have the llvm profiling runtime (I'll need to look into this I'll make another issue for it later). I'd recommend instead installing tarpaulin via something like https://github.com/cargo-bins/cargo-binstall if you want to cut down on CI runtime instead in the meantime |
I see. |
I ran into a similar issue link when trying to collect coverage in my workspace, when running tarpaulin in one package it would work fine but then another package would fail with the issue in the link. I was able to get around this using the Thought I'd share here in case it helps anyone else since I have nothing but a very vague assumptions as to what the actual underlying problem is. |
Anyone figured out another workaround for this, the |
@planck-length if you have an open source project I can check I can take a look at it |
Don't have an open source project but I can provide you with more info, let me know what you would like to know. I also ran previous command with |
Ok this one is resolved by updating libraries to latest, we're back on the previous one. For refference here's my cargo.toml
|
Ok my bad, I forgot to mention
So without |
Going to close this issue as resolved as there's been no further requests RE pyo3 support |
Not sure if this is a feature request or a bug - but am entering it into the bug system.
Happy to provide more information / help anyway I can.
Bug Description
cargo-tarpaulin
will fail with linking errors if run on a pyo3 project.Attached the full error here tarpaulin_error.txt
Reproducing
This can be reproduced by just creating a new pyo3 project with maturin
maturin new example_pyo3 cd example_pyo3 cargo-tarpaulin
Expected Behavior
I was expecting this to work like purely rust projects.
The text was updated successfully, but these errors were encountered: