| Message ID | 20210303124222.1485332-1-philmd@redhat.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | docker: Base Fedora MinGW cross containers on the base Fedora image | expand |
On Wed, Mar 03, 2021 at 01:42:22PM +0100, Philippe Mathieu-Daudé wrote: > The only difference between fedora-winXX-cross.docker and > fedora.docker is the specific QEMU_CONFIGURE_OPTS environment > variable. All the rest can be inherited from the generic Fedora > image. This is relying on the base Fedora image already having the mingw packages installed. My series to automate creation of the container dockerfiles removes those entirely, so we have strictly separate native and mingw images for Fedora: https://lists.gnu.org/archive/html/qemu-devel/2021-01/msg03123.html > Basing the cross containers on the generic Fedora allows us to > - save registry/local storage > - use more tools in the cross container (ccache for example). > > Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com> > --- > .gitlab-ci.d/containers.yml | 4 ++ > tests/docker/Makefile.include | 2 + > .../dockerfiles/fedora-win32-cross.docker | 41 +------------------ > .../dockerfiles/fedora-win64-cross.docker | 37 +---------------- > 4 files changed, 10 insertions(+), 74 deletions(-) Regards, Daniel
On 3/3/21 1:46 PM, Daniel P. Berrangé wrote: > On Wed, Mar 03, 2021 at 01:42:22PM +0100, Philippe Mathieu-Daudé wrote: >> The only difference between fedora-winXX-cross.docker and >> fedora.docker is the specific QEMU_CONFIGURE_OPTS environment >> variable. All the rest can be inherited from the generic Fedora >> image. > > This is relying on the base Fedora image already having the > mingw packages installed. My series to automate creation of > the container dockerfiles removes those entirely, so we have > strictly separate native and mingw images for Fedora: > > https://lists.gnu.org/archive/html/qemu-devel/2021-01/msg03123.html I'm a bit confused. So instead of having one single big image, you prefer to have multiple ones. I stopped using the fedora-winXX-cross because they lacks ccache, and the "big" fedora image contains the mingwXX toolchains. I'll wait you respin your series and test the new created containers to see if they fit my needs. Thanks, Phil.
On Thu, Mar 04, 2021 at 03:00:04PM +0100, Philippe Mathieu-Daudé wrote: > On 3/3/21 1:46 PM, Daniel P. Berrangé wrote: > > On Wed, Mar 03, 2021 at 01:42:22PM +0100, Philippe Mathieu-Daudé wrote: > >> The only difference between fedora-winXX-cross.docker and > >> fedora.docker is the specific QEMU_CONFIGURE_OPTS environment > >> variable. All the rest can be inherited from the generic Fedora > >> image. > > > > This is relying on the base Fedora image already having the > > mingw packages installed. My series to automate creation of > > the container dockerfiles removes those entirely, so we have > > strictly separate native and mingw images for Fedora: > > > > https://lists.gnu.org/archive/html/qemu-devel/2021-01/msg03123.html > > I'm a bit confused. So instead of having one single big image, > you prefer to have multiple ones. > > I stopped using the fedora-winXX-cross because they lacks ccache, > and the "big" fedora image contains the mingwXX toolchains. > > I'll wait you respin your series and test the new created containers > to see if they fit my needs. ccache will be present in any of the containers I auto-generate for QEMU using lcitool, as I agree its a very useful part o fthe toolchain. So when i convert the winXXX-cross containers, they'll gain ccache (and alot of other missing mingw pieces) Regards, Daniel
On 3/4/21 3:18 PM, Daniel P. Berrangé wrote: > On Thu, Mar 04, 2021 at 03:00:04PM +0100, Philippe Mathieu-Daudé wrote: >> On 3/3/21 1:46 PM, Daniel P. Berrangé wrote: >>> On Wed, Mar 03, 2021 at 01:42:22PM +0100, Philippe Mathieu-Daudé wrote: >>>> The only difference between fedora-winXX-cross.docker and >>>> fedora.docker is the specific QEMU_CONFIGURE_OPTS environment >>>> variable. All the rest can be inherited from the generic Fedora >>>> image. >>> >>> This is relying on the base Fedora image already having the >>> mingw packages installed. My series to automate creation of >>> the container dockerfiles removes those entirely, so we have >>> strictly separate native and mingw images for Fedora: >>> >>> https://lists.gnu.org/archive/html/qemu-devel/2021-01/msg03123.html >> >> I'm a bit confused. So instead of having one single big image, >> you prefer to have multiple ones. >> >> I stopped using the fedora-winXX-cross because they lacks ccache, >> and the "big" fedora image contains the mingwXX toolchains. >> >> I'll wait you respin your series and test the new created containers >> to see if they fit my needs. > > ccache will be present in any of the containers I auto-generate for > QEMU using lcitool, as I agree its a very useful part o fthe toolchain. > So when i convert the winXXX-cross containers, they'll gain ccache > (and alot of other missing mingw pieces) Cool, looking forward to QEMU using lcitool :)
diff --git a/.gitlab-ci.d/containers.yml b/.gitlab-ci.d/containers.yml index 33e4046e233..1f7f82ece65 100644 --- a/.gitlab-ci.d/containers.yml +++ b/.gitlab-ci.d/containers.yml @@ -222,11 +222,15 @@ i386-fedora-cross-container: win32-fedora-cross-container: <<: *container_job_definition + stage: containers-layer2 + needs: ['amd64-fedora-container'] variables: NAME: fedora-win32-cross win64-fedora-cross-container: <<: *container_job_definition + stage: containers-layer2 + needs: ['amd64-fedora-container'] variables: NAME: fedora-win64-cross diff --git a/tests/docker/Makefile.include b/tests/docker/Makefile.include index 7cab761bf5b..5aa99df32ad 100644 --- a/tests/docker/Makefile.include +++ b/tests/docker/Makefile.include @@ -144,6 +144,8 @@ docker-image-debian-riscv64-cross: docker-image-debian10 docker-image-debian-s390x-cross: docker-image-debian10 docker-image-debian-sh4-cross: docker-image-debian10 docker-image-debian-sparc64-cross: docker-image-debian10 +docker-image-fedora-win32-cross: docker-image-fedora +docker-image-fedora-win64-cross: docker-image-fedora # Specialist build images, sometimes very limited tools docker-image-debian-tricore-cross: docker-image-debian10 diff --git a/tests/docker/dockerfiles/fedora-win32-cross.docker b/tests/docker/dockerfiles/fedora-win32-cross.docker index 81b5659e9c5..425f05c3e98 100644 --- a/tests/docker/dockerfiles/fedora-win32-cross.docker +++ b/tests/docker/dockerfiles/fedora-win32-cross.docker @@ -1,41 +1,4 @@ -FROM fedora:33 - -# Please keep this list sorted alphabetically -ENV PACKAGES \ - bc \ - bzip2 \ - diffutils \ - findutils \ - gcc \ - gettext \ - git \ - hostname \ - make \ - meson \ - mingw32-bzip2 \ - mingw32-curl \ - mingw32-glib2 \ - mingw32-gmp \ - mingw32-gnutls \ - mingw32-gtk3 \ - mingw32-libjpeg-turbo \ - mingw32-libpng \ - mingw32-libtasn1 \ - mingw32-nettle \ - mingw32-nsis \ - mingw32-pixman \ - mingw32-pkg-config \ - mingw32-SDL2 \ - perl \ - perl-Test-Harness \ - python3 \ - python3-PyYAML \ - tar \ - which - -RUN dnf install -y $PACKAGES -RUN rpm -q $PACKAGES | sort > /packages.txt -ENV FEATURES mingw +FROM qemu/fedora # Specify the cross prefix for this image (see tests/docker/common.rc) -ENV QEMU_CONFIGURE_OPTS --cross-prefix=i686-w64-mingw32- +ENV QEMU_CONFIGURE_OPTS $QEMU_CONFIGURE_OPTS --cross-prefix=i686-w64-mingw32- diff --git a/tests/docker/dockerfiles/fedora-win64-cross.docker b/tests/docker/dockerfiles/fedora-win64-cross.docker index bcb428e7242..abdbce2a145 100644 --- a/tests/docker/dockerfiles/fedora-win64-cross.docker +++ b/tests/docker/dockerfiles/fedora-win64-cross.docker @@ -1,37 +1,4 @@ -FROM fedora:33 - -# Please keep this list sorted alphabetically -ENV PACKAGES \ - bc \ - bzip2 \ - diffutils \ - findutils \ - gcc \ - gettext \ - git \ - hostname \ - make \ - meson \ - mingw64-bzip2 \ - mingw64-curl \ - mingw64-glib2 \ - mingw64-gmp \ - mingw64-gtk3 \ - mingw64-libjpeg-turbo \ - mingw64-libpng \ - mingw64-libtasn1 \ - mingw64-pixman \ - mingw64-pkg-config \ - perl \ - perl-Test-Harness \ - python3 \ - python3-PyYAML \ - tar \ - which - -RUN dnf install -y $PACKAGES -RUN rpm -q $PACKAGES | sort > /packages.txt -ENV FEATURES mingw +FROM qemu/fedora # Specify the cross prefix for this image (see tests/docker/common.rc) -ENV QEMU_CONFIGURE_OPTS --cross-prefix=x86_64-w64-mingw32- --disable-capstone +ENV QEMU_CONFIGURE_OPTS $QEMU_CONFIGURE_OPTS --cross-prefix=x86_64-w64-mingw32- --disable-capstone
The only difference between fedora-winXX-cross.docker and fedora.docker is the specific QEMU_CONFIGURE_OPTS environment variable. All the rest can be inherited from the generic Fedora image. Basing the cross containers on the generic Fedora allows us to - save registry/local storage - use more tools in the cross container (ccache for example). Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com> --- .gitlab-ci.d/containers.yml | 4 ++ tests/docker/Makefile.include | 2 + .../dockerfiles/fedora-win32-cross.docker | 41 +------------------ .../dockerfiles/fedora-win64-cross.docker | 37 +---------------- 4 files changed, 10 insertions(+), 74 deletions(-)