Skip to content

no-panic currently cannot be used in doctests #28

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

Open
untitaker opened this issue Feb 6, 2022 · 0 comments
Open

no-panic currently cannot be used in doctests #28

untitaker opened this issue Feb 6, 2022 · 0 comments

Comments

@untitaker
Copy link

untitaker commented Feb 6, 2022

Attempting to use no-panic in a library requires the library to be used somehow, somewhere. The obvious answer to that is tests. For that, tests need to be compiled with opt-level=1 (as per README's dev profile). However, when the tests are run, it seems that while the regular integration tests do pass, the doctests fail with a linker error.

This is an artifact of rust-lang/cargo#4251

I think no-panic README should point this out at least, but IMO ideally it would also not even suggest to tweak the dev-profile but only suggest the optional dependency solution, as everything else seems to be a dead end.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant