-
Notifications
You must be signed in to change notification settings - Fork 190
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
Prebuilt image enhancements #365
Prebuilt image enhancements #365
Conversation
Signed-off-by: Jiaxun Yang <[email protected]>
Amazing contribution.. thanks a lot. I will cherry-pick to other branches |
@@ -27,6 +27,8 @@ | |||
FROM ubuntu:focal | |||
|
|||
ENV DEBIAN_FRONTEND=noninteractive | |||
ENV BUILD_UHD_FROM_SOURCE=True | |||
ENV UHD_VERSION=4.4.0.0 |
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.
Is there a possibility to not hardcode UHD version??
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.
Seems no, the build script requires this variable to be set.
4.4 was chosen by various official docker images, see: https://gitlab.eurecom.fr/oai/openairinterface5g/-/blob/develop/docker/Dockerfile.base.ubuntu22?ref_type=heads#L34
0e42a21
to
c650fdd
Compare
Upgrade to Ubuntu jammy so we have new limesuite and soapysdr in system repo. Use fixed v2.1 base. Build UHD from source so that TDD patch will be applied. Signed-off-by: Jiaxun Yang <[email protected]>
Signed-off-by: Jiaxun Yang <[email protected]>
c650fdd
to
c739b9d
Compare
@FlyGoat Hey, I will merge this once I am back from vacation |
Build all image variants, fix OAI image build, and provide document instructions.
Maybe it's worthy to cherry-pick those changes to all branches, so images will be built for all configurations.