Skip to content

Backport #809 to 0.11.x #816

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
wants to merge 5 commits into
base: release-11.x
Choose a base branch
from

Conversation

apoelstra
Copy link
Member

Backport of #809 to 11.x. Last backport for this PR.

apoelstra added 5 commits May 3, 2025 12:46
None of these belong in lib.rs, but these in particular have been
changed in rustc so that they now trigger on code that used to be okay.
This prevents me from running tests with recent rustcs on backports.
Just runs `./generate-files.sh` after tweaking it to sort the list.
Otherwise my local CI chokes on the inconsistency between different
'find' orderings.
When parsing a descriptor with `Descriptor::parse_descriptor`, we first parse
as a string and then parse the keys. We fail to consider parsing errors
in the keys, resulting in a panic.

Also, clean up the panic message so it's clearer what's going on.
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

Successfully merging this pull request may close these issues.

1 participant