Don't ignore transitive imports when probing a static library#77
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #77 +/- ##
=======================================
Coverage ? 75.14%
=======================================
Files ? 3
Lines ? 3022
Branches ? 0
=======================================
Hits ? 2271
Misses ? 751
Partials ? 0 ☔ View full report in Codecov by Sentry. |
|
We stopped relying on Shouldn't we fix that manual code instead then? |
d3dc609 to
c8b1873
Compare
|
@gdesmott I found the code, it needs a copy of rust-lang/pkg-config-rs#154 applied. Let me know if this looks OK now. |
c8b1873 to
a559417
Compare
|
@gdesmott Hi again, just updated the PR. Could this be reviewed now? |
a559417 to
e48632e
Compare
|
The change seems safe enough now. Which actual use case is it fixing? Did you check it build and is usable now? |
|
FWIW the pkg-config-rs change that does basically the same is merged and released |
e48632e to
c888d16
Compare
|
@amyspark : sorry I missed your latest update. Can you please rebase on top of |
Suppressing cargo_metadata silences libraries with full paths, in turn causing linking against static libraries to fail utterly on Windows.
c888d16 to
a89df41
Compare
|
@gdesmott done, let me know if anything else's needed. |
|
Released in |
This is needed towards building GStreamer's Rust plugins statically on Windows.
CC @nirbheek