[PULL 00/14] Gitlab-CI improvements and related fixes

Thomas Huth posted 14 patches 3 years, 7 months ago
Test docker-quick@centos7 failed
Test docker-mingw@fedora failed
Test checkpatch failed
Test FreeBSD failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20200907121127.136243-1-thuth@redhat.com
Maintainers: Xie Changlong <xiechanglong.d@gmail.com>, "Edgar E. Iglesias" <edgar.iglesias@gmail.com>, "Daniel P. Berrangé" <berrange@redhat.com>, Wen Congyang <wencongyang2@huawei.com>
.gitlab-ci.d/crossbuilds.yml                       | 113 +++++++++++++++++++++
.gitlab-ci.yml                                     |  69 +++++++++++++
MAINTAINERS                                        |   1 +
configure                                          |   3 +
crypto/meson.build                                 |  42 ++++++--
meson.build                                        |   5 +
stubs/cpu-get-icount.c                             |   5 -
stubs/fd-register.c                                |   6 --
stubs/meson.build                                  |   3 +-
stubs/notify-event.c                               |   6 --
stubs/qemu-timer-notify-cb.c                       |   8 ++
target/microblaze/cpu.c                            |  11 ++
target/microblaze/cpu.h                            |   3 +-
target/microblaze/helper.c                         |  17 ++--
target/microblaze/translate.c                      |   4 +-
tests/Makefile.include                             |   2 +
tests/docker/dockerfiles/centos7.docker            |   2 +
tests/docker/dockerfiles/centos8.docker            |   1 +
tests/docker/dockerfiles/debian-win64-cross.docker |   9 +-
tests/docker/dockerfiles/debian10.docker           |   1 +
tests/docker/dockerfiles/debian9.docker            |   1 +
tests/qtest/meson.build                            |   1 -
tests/test-replication.c                           |   4 +
tests/test-util-filemonitor.c                      |   4 +-
tests/test-vmstate.c                               |   4 +-
util/main-loop.c                                   |   4 +
26 files changed, 285 insertions(+), 44 deletions(-)
create mode 100644 .gitlab-ci.d/crossbuilds.yml
delete mode 100644 stubs/fd-register.c
delete mode 100644 stubs/notify-event.c
create mode 100644 stubs/qemu-timer-notify-cb.c
[PULL 00/14] Gitlab-CI improvements and related fixes
Posted by Thomas Huth 3 years, 7 months ago
 Hi Peter,

the following changes since commit 7c37270b3fbe3d034ba80e488761461676e21eb4:

  Merge remote-tracking branch 'remotes/kraxel/tags/ui-20200904-pull-request' into staging (2020-09-06 16:23:55 +0100)

are available in the Git repository at:

  https://gitlab.com/huth/qemu.git tags/pull-request-2020-09-07

for you to fetch changes up to 88e74b6122bb40852b1c98befd47a572f0c06930:

  target/microblaze: Collected fixes for env->iflags (2020-09-07 12:34:17 +0200)

----------------------------------------------------------------
* Fixes for cross-compiling with MinGW / compiling with MSYS2
* Enable cross-compiler builds in the Gitlab CI
* Improvements / fixes for the crypto tests in the Gitlab CI
* Fix for the "make check-acceptance" microblaze failure
  (required to get the Gitlab CI green again)
----------------------------------------------------------------

Alex Bennée (1):
      tests/docker: add python3-setuptools the docker images

Daniel P. Berrangé (2):
      crypto: fix build with gcrypt enabled
      gitlab: expand test coverage for crypto builds

Paolo Bonzini (1):
      tests/qtest: do not list ahci-test twice

Richard Henderson (1):
      target/microblaze: Collected fixes for env->iflags

Thomas Huth (6):
      tests/Makefile: test-image-locking needs CONFIG_POSIX
      dockerfiles/debian-win64-cross: Download WHPX MinGW headers
      configure: Allow automatic WHPX detection
      stubs: Move qemu_timer_notify_cb() and remove qemu_notify_event() stub
      stubs: Move qemu_fd_register stub to util/main-loop.c
      gitlab-ci: Add cross-compiling build tests

Yonggang Luo (3):
      tests: handling signal on win32 properly
      tests: fixes test-vmstate.c compile error on msys2
      tests: Fixes building test-util-filemonitor.c on msys2/mingw

 .gitlab-ci.d/crossbuilds.yml                       | 113 +++++++++++++++++++++
 .gitlab-ci.yml                                     |  69 +++++++++++++
 MAINTAINERS                                        |   1 +
 configure                                          |   3 +
 crypto/meson.build                                 |  42 ++++++--
 meson.build                                        |   5 +
 stubs/cpu-get-icount.c                             |   5 -
 stubs/fd-register.c                                |   6 --
 stubs/meson.build                                  |   3 +-
 stubs/notify-event.c                               |   6 --
 stubs/qemu-timer-notify-cb.c                       |   8 ++
 target/microblaze/cpu.c                            |  11 ++
 target/microblaze/cpu.h                            |   3 +-
 target/microblaze/helper.c                         |  17 ++--
 target/microblaze/translate.c                      |   4 +-
 tests/Makefile.include                             |   2 +
 tests/docker/dockerfiles/centos7.docker            |   2 +
 tests/docker/dockerfiles/centos8.docker            |   1 +
 tests/docker/dockerfiles/debian-win64-cross.docker |   9 +-
 tests/docker/dockerfiles/debian10.docker           |   1 +
 tests/docker/dockerfiles/debian9.docker            |   1 +
 tests/qtest/meson.build                            |   1 -
 tests/test-replication.c                           |   4 +
 tests/test-util-filemonitor.c                      |   4 +-
 tests/test-vmstate.c                               |   4 +-
 util/main-loop.c                                   |   4 +
 26 files changed, 285 insertions(+), 44 deletions(-)
 create mode 100644 .gitlab-ci.d/crossbuilds.yml
 delete mode 100644 stubs/fd-register.c
 delete mode 100644 stubs/notify-event.c
 create mode 100644 stubs/qemu-timer-notify-cb.c


Re: [PULL 00/14] Gitlab-CI improvements and related fixes
Posted by 罗勇刚 (Yonggang Luo) 3 years, 7 months ago
On Mon, Sep 7, 2020 at 8:11 PM Thomas Huth <thuth@redhat.com> wrote:

>  Hi Peter,
>
> the following changes since commit
> 7c37270b3fbe3d034ba80e488761461676e21eb4:
>
>   Merge remote-tracking branch
> 'remotes/kraxel/tags/ui-20200904-pull-request' into staging (2020-09-06
> 16:23:55 +0100)
>
> are available in the Git repository at:
>
>   https://gitlab.com/huth/qemu.git tags/pull-request-2020-09-07
>
> for you to fetch changes up to 88e74b6122bb40852b1c98befd47a572f0c06930:
>
>   target/microblaze: Collected fixes for env->iflags (2020-09-07 12:34:17
> +0200)
>
> ----------------------------------------------------------------
> * Fixes for cross-compiling with MinGW / compiling with MSYS2
> * Enable cross-compiler builds in the Gitlab CI
> * Improvements / fixes for the crypto tests in the Gitlab CI
> * Fix for the "make check-acceptance" microblaze failure
>   (required to get the Gitlab CI green again)
> ----------------------------------------------------------------
>
> Alex Bennée (1):
>       tests/docker: add python3-setuptools the docker images
>
> Daniel P. Berrangé (2):
>       crypto: fix build with gcrypt enabled
>       gitlab: expand test coverage for crypto builds
>
> Paolo Bonzini (1):
>       tests/qtest: do not list ahci-test twice
>
> Richard Henderson (1):
>       target/microblaze: Collected fixes for env->iflags
>
> Thomas Huth (6):
>       tests/Makefile: test-image-locking needs CONFIG_POSIX
>       dockerfiles/debian-win64-cross: Download WHPX MinGW headers
>       configure: Allow automatic WHPX detection
>       stubs: Move qemu_timer_notify_cb() and remove qemu_notify_event()
> stub
>       stubs: Move qemu_fd_register stub to util/main-loop.c
>       gitlab-ci: Add cross-compiling build tests
>
> Yonggang Luo (3):
>       tests: handling signal on win32 properly
>
Hi thomas, this have already been queued by Bonzini, you have to skip this.

>       tests: fixes test-vmstate.c compile error on msys2
>
Hi Bonzini, the int compiling error are fixed by this commit

>       tests: Fixes building test-util-filemonitor.c on msys2/mingw
>
Hi Bonzini, the mkdir error are fixed by this commit

>
>  .gitlab-ci.d/crossbuilds.yml                       | 113
> +++++++++++++++++++++
>  .gitlab-ci.yml                                     |  69 +++++++++++++
>  MAINTAINERS                                        |   1 +
>  configure                                          |   3 +
>  crypto/meson.build                                 |  42 ++++++--
>  meson.build                                        |   5 +
>  stubs/cpu-get-icount.c                             |   5 -
>  stubs/fd-register.c                                |   6 --
>  stubs/meson.build                                  |   3 +-
>  stubs/notify-event.c                               |   6 --
>  stubs/qemu-timer-notify-cb.c                       |   8 ++
>  target/microblaze/cpu.c                            |  11 ++
>  target/microblaze/cpu.h                            |   3 +-
>  target/microblaze/helper.c                         |  17 ++--
>  target/microblaze/translate.c                      |   4 +-
>  tests/Makefile.include                             |   2 +
>  tests/docker/dockerfiles/centos7.docker            |   2 +
>  tests/docker/dockerfiles/centos8.docker            |   1 +
>  tests/docker/dockerfiles/debian-win64-cross.docker |   9 +-
>  tests/docker/dockerfiles/debian10.docker           |   1 +
>  tests/docker/dockerfiles/debian9.docker            |   1 +
>  tests/qtest/meson.build                            |   1 -
>  tests/test-replication.c                           |   4 +
>  tests/test-util-filemonitor.c                      |   4 +-
>  tests/test-vmstate.c                               |   4 +-
>  util/main-loop.c                                   |   4 +
>  26 files changed, 285 insertions(+), 44 deletions(-)
>  create mode 100644 .gitlab-ci.d/crossbuilds.yml
>  delete mode 100644 stubs/fd-register.c
>  delete mode 100644 stubs/notify-event.c
>  create mode 100644 stubs/qemu-timer-notify-cb.c
>
>

-- 
         此致
礼
罗勇刚
Yours
    sincerely,
Yonggang Luo
Re: [PULL 00/14] Gitlab-CI improvements and related fixes
Posted by Peter Maydell 3 years, 7 months ago
On Mon, 7 Sep 2020 at 13:11, Thomas Huth <thuth@redhat.com> wrote:
>
>  Hi Peter,
>
> the following changes since commit 7c37270b3fbe3d034ba80e488761461676e21eb4:
>
>   Merge remote-tracking branch 'remotes/kraxel/tags/ui-20200904-pull-request' into staging (2020-09-06 16:23:55 +0100)
>
> are available in the Git repository at:
>
>   https://gitlab.com/huth/qemu.git tags/pull-request-2020-09-07
>
> for you to fetch changes up to 88e74b6122bb40852b1c98befd47a572f0c06930:
>
>   target/microblaze: Collected fixes for env->iflags (2020-09-07 12:34:17 +0200)
>
> ----------------------------------------------------------------
> * Fixes for cross-compiling with MinGW / compiling with MSYS2
> * Enable cross-compiler builds in the Gitlab CI
> * Improvements / fixes for the crypto tests in the Gitlab CI
> * Fix for the "make check-acceptance" microblaze failure
>   (required to get the Gitlab CI green again)
> ----------------------------------------------------------------


Applied, thanks.

Please update the changelog at https://wiki.qemu.org/ChangeLog/5.2
for any user-visible changes.

-- PMM

Re: [PULL 00/14] Gitlab-CI improvements and related fixes
Posted by 罗勇刚 (Yonggang Luo) 3 years, 7 months ago
On Tue, Sep 8, 2020 at 2:26 AM Peter Maydell <peter.maydell@linaro.org>
wrote:

> On Mon, 7 Sep 2020 at 13:11, Thomas Huth <thuth@redhat.com> wrote:
> >
> >  Hi Peter,
> >
> > the following changes since commit
> 7c37270b3fbe3d034ba80e488761461676e21eb4:
> >
> >   Merge remote-tracking branch
> 'remotes/kraxel/tags/ui-20200904-pull-request' into staging (2020-09-06
> 16:23:55 +0100)
> >
> > are available in the Git repository at:
> >
> >   https://gitlab.com/huth/qemu.git tags/pull-request-2020-09-07
> >
> > for you to fetch changes up to 88e74b6122bb40852b1c98befd47a572f0c06930:
> >
> >   target/microblaze: Collected fixes for env->iflags (2020-09-07
> 12:34:17 +0200)
> >
> > ----------------------------------------------------------------
> > * Fixes for cross-compiling with MinGW / compiling with MSYS2
> > * Enable cross-compiler builds in the Gitlab CI
> > * Improvements / fixes for the crypto tests in the Gitlab CI
> > * Fix for the "make check-acceptance" microblaze failure
> >   (required to get the Gitlab CI green again)
> > ----------------------------------------------------------------
>
>
> Applied, thanks.
>
> Please update the changelog at https://wiki.qemu.org/ChangeLog/5.2
> for any user-visible changes.
>
> -- PMM
>


-- 
         此致
礼
罗勇刚
Yours
    sincerely,
Yonggang Luo