-
Notifications
You must be signed in to change notification settings - Fork 453
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
Fix #2399 - duplication error with __v1model_version in v1model #2398
Conversation
You don't need to include the json filename in double-quotes. I am using a p4c as latest as from yesterday and I cannot reproduce the issue. The P4 program I have used also includes v1model.p4.
|
Update your p4 compiler again. The commit that introduced this issue was from 16 hours ago. Check then. |
After updating my p4c to latest, I can reproduce the issue, thanks. |
This checkin has caused the issue: 93f4598#diff-68d536ca6ecb3bfe9ddbb6d67f5a6207 Please wait for @mbudiu-vmw to review this PR code changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like an auto-merge snafu
We should make sure we keep the one in the middle. |
After fixing this, please also update the p4c docker image, since it also contains this bug. |
The docker image will be updated automatically after a while. |
When you try to compile using
p4c-bm2-ss --p4v 16 "foo.p4" -o "bar.json"
,where
foo.p4
includesv1model.p4
, it throws this error -This because __v1model_version is declared twice. This PR fixes that by, well, removing a line. Felt it would be better to just create a PR than creating an issue. This fixes issue #2399