When enabled, PINS/p4rt is not generated, it is generated only for bullseye, hence c version errors #10127
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
PINS only runs on buster and because of this bug, it does not run on buster and only on bullseye. Related to issue number: sonic-net#9885
mfyuce
changed the title
mfyuce
changed the title
Added required libgmp-dev dependency for p4rt binary running in docker-sonic-p4rt container.
|
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
libgmpxx.so.4dependency")Related to issue : #9885
Why I did it
I could not build SONIC with p4rt enabled. When I succeeded building, generated docker would be buster but the p4rt bin inside would be bullseye.
How I did it
Added two line to the respective p4rt make file
How to verify it
Follow PINS build instructions at here (in current state with master it is not working without this fix)
Which release branch to backport (provide reason below if selected)
202111: Because on the public anouncements, PINS said to be entegrated to this branch
Description for the changelog
Link to config_db schema for YANG module changes
A picture of a cute animal (not mandatory but encouraged)
:)
