search

microsoft / mu_devops

Project Mu Developer Operations
https://microsoft.github.io/mu/
Other
27 stars 25 forks source link

.sync/Version.njk: Switch to Project Mu Ubuntu 22 containers #182

Closed makubacki closed 1 year ago

makubacki commented 1 year ago

Updates the default Linux build container from Fedora 37 to Ubuntu 22.

The container feed switched from ghcr.io/tianocore/containers to ghcr.io/microsoft/mu_devops.

Note: This is marked as a breaking change because of the distro change from Fedora to Ubuntu. Commands run inside the container such as those interacting with the package manager need to be reviewed and updated.

makubacki commented 1 year ago

Looks like gcc-mingw-w64 might be needed instead of mingw64-gcc. Is that what you've used on Ubuntu?

gcc-mingw-w64 - GNU C compiler for MinGW-w64
gcc-mingw-w64-base - GNU Compiler Collection for MinGW-w64 (base package)
gcc-mingw-w64-i686 - GNU C compiler for MinGW-w64 targeting Win32
gcc-mingw-w64-x86-64 - GNU C compiler for MinGW-w64 targeting Win64
kuqin12 commented 1 year ago

Looks like gcc-mingw-w64 might be needed instead of mingw64-gcc. Is that what you've used on Ubuntu?

gcc-mingw-w64 - GNU C compiler for MinGW-w64
gcc-mingw-w64-base - GNU Compiler Collection for MinGW-w64 (base package)
gcc-mingw-w64-i686 - GNU C compiler for MinGW-w64 targeting Win32
gcc-mingw-w64-x86-64 - GNU C compiler for MinGW-w64 targeting Win64

I don't see it being used when I did the overhaul on the other PR https://github.com/microsoft/mu_devops/pull/154. Should we just remove this step?

makubacki commented 1 year ago

I don't see it being used when I did the overhaul on the other PR #154. Should we just remove this step?

@cfernald - Do you know if this is actually needed? I can test removing it and running on a pipeline if we need that data.

makubacki commented 1 year ago

I don't see it being used when I did the overhaul on the other PR #154. Should we just remove this step?

@cfernald - Do you know if this is actually needed? I can test removing it and running on a pipeline if we need that data.

We discussed offline. I'm just going to remove it for now since that has been previously tested with no issues observed.