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
'features' flags does not work #20
Comments
Thanks for the report! I actually did try them on your resvg crate and it seemed to me like it worked as expected. Can you elaborate a bit on "does not work"? :) |
If you set |
I see the dependencies, pango, cairo and resvg-qt, they are scanned but "not used"...
Is this different from your results? I'm on macOS right now. |
I have an error, so I don't know. According to the log in https://github.com/anderejd/cargo-geiger/issues/19, there are no pango, cairo, etc. |
Do you get the panic when running I suspect that this issue will need to wait until the error handling is sorted out the the panics are resolved. |
Now I see it. Thanks. |
But I still have |
Yeah, I see those warnings here too. They seems to be identical to that warnings that show up when using I don't know if the warnings would go away on this system when adding |
Ah... Now I get it. Yes. it seems like the normal cargo is doing something different with --all-features. I suspect that the build scripts could be involved in this issue. I'll open new issue for build scripts and reference this issue. |
This error is reported by
This is expected since this system doesn't have QT_DIR. However, when running |
|
Yes, I did expect the same error to appear when running |
My own |
Yes I agree, I think I'm just failing to express myself in the last ~3 comments. I opened a new issue for suspected build script bugs in cargo-geiger and referenced this discussion as background. |
The original issue was correct all along, the feature flags was not being used correctly. Fixed in master at: anderejd@33771aa Thanks for the assistance! :) I still think the build scripts needs to be investigated, but for different reasons, to scan the generated code from bindgen for example. |
This one:
The text was updated successfully, but these errors were encountered: