[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PULL 00/10] Gitlab-CI, qtest, moxie removal and misc patches
From: |
Peter Maydell |
Subject: |
Re: [PULL 00/10] Gitlab-CI, qtest, moxie removal and misc patches |
Date: |
Thu, 6 May 2021 08:38:42 +0100 |
On Thu, 6 May 2021 at 08:01, Thomas Huth <thuth@redhat.com> wrote:
>
> On 05/05/2021 20.06, Peter Maydell wrote:
> > On Mon, 3 May 2021 at 11:45, Thomas Huth <thuth@redhat.com> wrote:
> >>
> >> Hi Peter,
> >>
> >> the following changes since commit
> >> 53c5433e84e8935abed8e91d4a2eb813168a0ecf:
> >>
> >> Merge remote-tracking branch
> >> 'remotes/rth-gitlab/tags/pull-tcg-20210501' into staging (2021-05-02
> >> 12:02:46 +0100)
> >>
> >> are available in the Git repository at:
> >>
> >> https://gitlab.com/thuth/qemu.git tags/pull-request-2021-05-03
> >>
> >> for you to fetch changes up to 8f582fa290e5d5d0a00db23eaf1ab1bb3d3ae68d:
> >>
> >> util/compatfd.c: Replaced a malloc call with g_malloc. (2021-05-03
> >> 11:40:40 +0200)
> >>
> >> ----------------------------------------------------------------
> >> * Removal of the deprecated moxie target
> >> * Replace some YAML anchors by "extends" in the Gitlab-CI
> >> * Some small improvements for using the qtests
> >> * Some other small misc patches
> >
> > This fails to build as an incremental (not from-clean) build:
> [...]
> > ../../meson.build:1291:2: ERROR: Failed to load
> > /home/ubuntu/qemu/default-configs/targets/moxie-softmmu.mak: [Errno 2]
> > No such file or directory:
> > '/home/ubuntu/qemu/default-configs/targets/moxie-softmmu.mak'
>
> D'oh! I think I can work-around the problem with a patch like
> this on top:
>
> diff a/configure b/configure
> --- a/configure
> +++ b/configure
> @@ -1686,6 +1686,11 @@ fi
>
> for config in $mak_wilds; do
> target="$(basename "$config" .mak)"
> + if [ "$target" = "moxie-softmmu" ]; then
> + # This is a work-around to make incremental builds pass after
> + # moxie-softmmu has been removed. It can be removed later.
> + continue
> + fi
> if echo "$target_list_exclude" | grep -vq "$target"; then
> default_target_list="${default_target_list} $target"
> fi
> diff a/default-configs/targets/moxie-softmmu.mak
> b/default-configs/targets/moxie-softmmu.mak
> new file mode 100644
> index 0000000000..23fd596b66
> --- /dev/null
> +++ b/default-configs/targets/moxie-softmmu.mak
> @@ -0,0 +1,2 @@
> +# This is just a dummy file to avoid that incremental builds are failing.
> +# It can be removed as soon as all builders have been updated.
>
> Does that look acceptable? If yes, I'll respin my PR with that
> squashed into the moxie patch.
Ugly, but I guess so. It would be nice to fix the underlying cause, though:
meson/ninja should just DTRT if we remove a target. We have a couple
of other target removals coming through as well...
thanks
-- PMM
- [PULL 01/10] Remove the deprecated moxie target, (continued)
- [PULL 01/10] Remove the deprecated moxie target, Thomas Huth, 2021/05/03
- [PULL 04/10] gitlab-ci: Replace YAML anchors by extends (container_job), Thomas Huth, 2021/05/03
- [PULL 05/10] gitlab-ci: Replace YAML anchors by extends (native_build_job), Thomas Huth, 2021/05/03
- [PULL 06/10] gitlab-ci: Replace YAML anchors by extends (native_test_job), Thomas Huth, 2021/05/03
- [PULL 07/10] libqos/qgraph: fix "UNAVAILBLE" typo, Thomas Huth, 2021/05/03
- [PULL 08/10] docs/devel/qgraph: add troubleshooting information, Thomas Huth, 2021/05/03
- [PULL 10/10] util/compatfd.c: Replaced a malloc call with g_malloc., Thomas Huth, 2021/05/03
- [PULL 09/10] libqtest: refuse QTEST_QEMU_BINARY=qemu-kvm, Thomas Huth, 2021/05/03
- Re: [PULL 00/10] Gitlab-CI, qtest, moxie removal and misc patches, Peter Maydell, 2021/05/05
- Re: [PULL 00/10] Gitlab-CI, qtest, moxie removal and misc patches, Thomas Huth, 2021/05/06
- Re: [PULL 00/10] Gitlab-CI, qtest, moxie removal and misc patches,
Peter Maydell <=
- Re: [PULL 00/10] Gitlab-CI, qtest, moxie removal and misc patches, Thomas Huth, 2021/05/07
- Re: [PULL 00/10] Gitlab-CI, qtest, moxie removal and misc patches, Paolo Bonzini, 2021/05/07
- Re: [PULL 00/10] Gitlab-CI, qtest, moxie removal and misc patches, Thomas Huth, 2021/05/09
- Re: [PULL 00/10] Gitlab-CI, qtest, moxie removal and misc patches, Philippe Mathieu-Daudé, 2021/05/14
- Re: [PULL 00/10] Gitlab-CI, qtest, moxie removal and misc patches, Thomas Huth, 2021/05/14
- Re: [PULL 00/10] Gitlab-CI, qtest, moxie removal and misc patches, Philippe Mathieu-Daudé, 2021/05/19
- Re: [PULL 00/10] Gitlab-CI, qtest, moxie removal and misc patches, Thomas Huth, 2021/05/20
- Re: [PULL 00/10] Gitlab-CI, qtest, moxie removal and misc patches, Paolo Bonzini, 2021/05/20
- Re: [PULL 00/10] Gitlab-CI, qtest, moxie removal and misc patches, Philippe Mathieu-Daudé, 2021/05/20
- Re: [PULL 00/10] Gitlab-CI, qtest, moxie removal and misc patches, Peter Maydell, 2021/05/20