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.

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

Initial check-in for MU Linux container and pipelines #175

Merged
merged 7 commits into from
May 24, 2023

Conversation

cfernald
Copy link
Contributor

Creates a Ubuntu-22 based image for building Project MU. These changes are largely based on the infrastructure from tianocore/containers.

Sizes:
Build - 2.74 Gb
Test - 3.63 Gb
Dev - 3.63 Gb

Containers/Readme.md Outdated Show resolved Hide resolved
@makubacki
Copy link
Member

@cfernald was there a particular reason for choosing Ubuntu?

I think it's fine, but it would be helpful to describe in the containers readme file why that distro was chosen for Mu Linux builds.

@cfernald
Copy link
Contributor Author

@cfernald was there a particular reason for choosing Ubuntu?

I think it's fine, but it would be helpful to describe in the containers readme file why that distro was chosen for Mu Linux builds.

2 reason, first Fedora has limited support cross-compiling usermode components which is problematic for basetools (which may not matter), but also people are just generally more familiar and aligns with peoples existing development flows better.

@makubacki
Copy link
Member

@cfernald was there a particular reason for choosing Ubuntu?
I think it's fine, but it would be helpful to describe in the containers readme file why that distro was chosen for Mu Linux builds.

2 reason, first Fedora has limited support cross-compiling usermode components which is problematic for basetools (which may not matter), but also people are just generally more familiar and aligns with peoples existing development flows better.

Please add this info to the containers readme file.

@cfernald cfernald merged commit 4a1f8d3 into microsoft:main May 24, 2023
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.

3 participants