Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Frontend panic on protobuf #1014

Closed
maximebuyse opened this issue Oct 21, 2024 · 2 comments
Closed

Frontend panic on protobuf #1014

maximebuyse opened this issue Oct 21, 2024 · 2 comments
Labels
bug Something isn't working frontend Issue in the Rust to JSON translation

Comments

@maximebuyse
Copy link
Contributor

Running cargo hax json on https://github.com/stepancheg/rust-protobuf/tree/master panics with:

thread 'rustc' panicked at /home/maxime/cryspen/hax/frontend/exporter/src/types/copied.rs:1378:60:
DefId::expect_local: `DefId(22:6447 ~ protobuf[e121]::wire_format::WireType)` isn't local
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
@maximebuyse maximebuyse added bug Something isn't working frontend Issue in the Rust to JSON translation labels Oct 21, 2024
@W95Psp
Copy link
Collaborator

W95Psp commented Oct 21, 2024

Thanks, that's a tiny bug, I have a fix, pushing a PR in a few minutes

W95Psp added a commit that referenced this issue Oct 21, 2024
@W95Psp
fix(frontend): issue #1014
3a5f108
github-merge-queue bot pushed a commit that referenced this issue Oct 21, 2024
@maximebuyse
Merge pull request #1015 from hacspec/fix-1014
5775fce 
fix(frontend): issue #1014
@W95Psp
Copy link
Collaborator

W95Psp commented Oct 21, 2024

Was fixed by #1015

@W95Psp W95Psp closed this as completed Oct 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working frontend Issue in the Rust to JSON translation
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@W95Psp @maximebuyse