| Message ID | 20200914150716.10501-6-alex.bennee@linaro.org |
|---|---|
| State | Superseded |
| Headers | show |
| Series | deprecation and linux-user tweaks (+test fix) | expand |
On 9/14/20 5:07 PM, Alex Bennée wrote: > These targets might be deprecated but we should keep them building > before the final axe comes down. Lets keep them all in one place and > don't hold up the CI if they do fail. They are either poorly tested or > already flaky anyway, > > Signed-off-by: Alex Bennée <alex.bennee@linaro.org> > --- > .gitlab-ci.yml | 10 ++++++++++ > .travis.yml | 3 +-- > 2 files changed, 11 insertions(+), 2 deletions(-) > > diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml > index 72e8604579ee..f027b55aef15 100644 > --- a/.gitlab-ci.yml > +++ b/.gitlab-ci.yml > @@ -254,6 +254,16 @@ build-clang: > ppc-softmmu s390x-softmmu arm-linux-user > MAKE_CHECK_ARGS: check > > +# These targets are on the way out > +build-deprecated: > + <<: *native_build_job_definition > + variables: > + IMAGE: debian-all-test-cross > + CONFIGURE_ARGS: --disable-docs --disable-tools --disable-system > + MAKE_CHECK_ARGS: check-tcg > + TARGETS: ppc64abi32-linux-user tilegx-linux-user > + allow_failure: true > + > build-oss-fuzz: > <<: *native_build_job_definition > variables: > diff --git a/.travis.yml b/.travis.yml > index 65341634d02a..c75221dca3e7 100644 > --- a/.travis.yml > +++ b/.travis.yml > @@ -344,10 +344,9 @@ jobs: > # Run check-tcg against linux-user (with plugins) > # we skip sparc64-linux-user until it has been fixed somewhat > # we skip cris-linux-user as it doesn't use the common run loop > - # we skip ppc64abi32-linux-user as it seems to have a broken libc > - name: "GCC plugins check-tcg (user)" > env: > - - CONFIG="--disable-system --enable-plugins --enable-debug-tcg --target-list-exclude=sparc64-linux-user,cris-linux-user,ppc64abi32-linux-user" Maybe split the patch in 2, first the Travis-CI removal, mentioning commit c609274b853 ("docs/system/deprecated: mark ppc64abi32-linux-user for deprecation"), then adding the build-deprecated to GitLab-CI in a different patch. Preferably split: Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com> > + - CONFIG="--disable-system --enable-plugins --enable-debug-tcg --target-list-exclude=sparc64-linux-user,cris-linux-user" > - TEST_BUILD_CMD="make build-tcg" > - TEST_CMD="make check-tcg" > - CACHE_NAME="${TRAVIS_BRANCH}-linux-gcc-debug-tcg" >
On 14/09/2020 17.07, Alex Bennée wrote: > These targets might be deprecated but we should keep them building > before the final axe comes down. Lets keep them all in one place and > don't hold up the CI if they do fail. They are either poorly tested or > already flaky anyway, > > Signed-off-by: Alex Bennée <alex.bennee@linaro.org> > --- > .gitlab-ci.yml | 10 ++++++++++ > .travis.yml | 3 +-- > 2 files changed, 11 insertions(+), 2 deletions(-) > > diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml > index 72e8604579ee..f027b55aef15 100644 > --- a/.gitlab-ci.yml > +++ b/.gitlab-ci.yml > @@ -254,6 +254,16 @@ build-clang: > ppc-softmmu s390x-softmmu arm-linux-user > MAKE_CHECK_ARGS: check > > +# These targets are on the way out > +build-deprecated: > + <<: *native_build_job_definition > + variables: > + IMAGE: debian-all-test-cross > + CONFIGURE_ARGS: --disable-docs --disable-tools --disable-system > + MAKE_CHECK_ARGS: check-tcg > + TARGETS: ppc64abi32-linux-user tilegx-linux-user > + allow_failure: true > + > build-oss-fuzz: > <<: *native_build_job_definition > variables: > diff --git a/.travis.yml b/.travis.yml > index 65341634d02a..c75221dca3e7 100644 > --- a/.travis.yml > +++ b/.travis.yml > @@ -344,10 +344,9 @@ jobs: > # Run check-tcg against linux-user (with plugins) > # we skip sparc64-linux-user until it has been fixed somewhat > # we skip cris-linux-user as it doesn't use the common run loop > - # we skip ppc64abi32-linux-user as it seems to have a broken libc > - name: "GCC plugins check-tcg (user)" > env: > - - CONFIG="--disable-system --enable-plugins --enable-debug-tcg --target-list-exclude=sparc64-linux-user,cris-linux-user,ppc64abi32-linux-user" > + - CONFIG="--disable-system --enable-plugins --enable-debug-tcg --target-list-exclude=sparc64-linux-user,cris-linux-user" > - TEST_BUILD_CMD="make build-tcg" > - TEST_CMD="make check-tcg" > - CACHE_NAME="${TRAVIS_BRANCH}-linux-gcc-debug-tcg" > Acked-by: Thomas Huth <thuth@redhat.com>
diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index 72e8604579ee..f027b55aef15 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -254,6 +254,16 @@ build-clang: ppc-softmmu s390x-softmmu arm-linux-user MAKE_CHECK_ARGS: check +# These targets are on the way out +build-deprecated: + <<: *native_build_job_definition + variables: + IMAGE: debian-all-test-cross + CONFIGURE_ARGS: --disable-docs --disable-tools --disable-system + MAKE_CHECK_ARGS: check-tcg + TARGETS: ppc64abi32-linux-user tilegx-linux-user + allow_failure: true + build-oss-fuzz: <<: *native_build_job_definition variables: diff --git a/.travis.yml b/.travis.yml index 65341634d02a..c75221dca3e7 100644 --- a/.travis.yml +++ b/.travis.yml @@ -344,10 +344,9 @@ jobs: # Run check-tcg against linux-user (with plugins) # we skip sparc64-linux-user until it has been fixed somewhat # we skip cris-linux-user as it doesn't use the common run loop - # we skip ppc64abi32-linux-user as it seems to have a broken libc - name: "GCC plugins check-tcg (user)" env: - - CONFIG="--disable-system --enable-plugins --enable-debug-tcg --target-list-exclude=sparc64-linux-user,cris-linux-user,ppc64abi32-linux-user" + - CONFIG="--disable-system --enable-plugins --enable-debug-tcg --target-list-exclude=sparc64-linux-user,cris-linux-user" - TEST_BUILD_CMD="make build-tcg" - TEST_CMD="make check-tcg" - CACHE_NAME="${TRAVIS_BRANCH}-linux-gcc-debug-tcg"
These targets might be deprecated but we should keep them building before the final axe comes down. Lets keep them all in one place and don't hold up the CI if they do fail. They are either poorly tested or already flaky anyway, Signed-off-by: Alex Bennée <alex.bennee@linaro.org> --- .gitlab-ci.yml | 10 ++++++++++ .travis.yml | 3 +-- 2 files changed, 11 insertions(+), 2 deletions(-)