diff mbox series

devtools: reduce list of Arm builds tested with meson

Message ID 20191120234222.23540-1-thomas@monjalon.net (mailing list archive)
State Accepted, archived
Headers
Series devtools: reduce list of Arm builds tested with meson |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation fail apply issues
ci/travis-robot success Travis build: passed

Commit Message

Thomas Monjalon Nov. 20, 2019, 11:42 p.m. UTC 
  The list of Arm configs is growing:
	config/arm/arm64_armada_linux_gcc
	config/arm/arm64_armv8_linux_gcc
	config/arm/arm64_bluefield_linux_gcc
	config/arm/arm64_dpaa_linux_gcc
	config/arm/arm64_emag_linux_gcc
	config/arm/arm64_n1sdp_linux_gcc
	config/arm/arm64_octeontx2_linux_gcc
	config/arm/arm64_thunderx2_linux_gcc
	config/arm/arm64_thunderx_linux_gcc

In order to keep testing time reasonable,
and also because n1sdp is merged without a related fix in tests,
the list of configs is reduced in the script test-meson-builds.sh.

The list of tested Arm builds becomes:
	build-arm64-host-clang (armv8a)
	build-arm64-bluefield
	build-arm64-dpaa
	build-arm64-octeontx2

Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
Acked-by: David Marchand <david.marchand@redhat.com>
---
 devtools/test-meson-builds.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Thomas Monjalon Nov. 20, 2019, 11:47 p.m. UTC | #1 
21/11/2019 00:42, Thomas Monjalon:
> The list of Arm configs is growing:
> 	config/arm/arm64_armada_linux_gcc
> 	config/arm/arm64_armv8_linux_gcc
> 	config/arm/arm64_bluefield_linux_gcc
> 	config/arm/arm64_dpaa_linux_gcc
> 	config/arm/arm64_emag_linux_gcc
> 	config/arm/arm64_n1sdp_linux_gcc
> 	config/arm/arm64_octeontx2_linux_gcc
> 	config/arm/arm64_thunderx2_linux_gcc
> 	config/arm/arm64_thunderx_linux_gcc
> 
> In order to keep testing time reasonable,
> and also because n1sdp is merged without a related fix in tests,
> the list of configs is reduced in the script test-meson-builds.sh.
> 
> The list of tested Arm builds becomes:
> 	build-arm64-host-clang (armv8a)
> 	build-arm64-bluefield
> 	build-arm64-dpaa
> 	build-arm64-octeontx2
> 
> Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
> Acked-by: David Marchand <david.marchand@redhat.com>

It is keeping one target per vendor (Arm, Marvell, Mellanox, NXP).
I hope it is enough.
When rebuilding all, we had 10 Arm targets (including the host-clang),
it is really too much in my opinion. It cuts from 10 to 4.
And first benefit is to skip n1sdp which we should fix properly in -rc4.

Applied quickly before closing -rc3.
diff mbox series

Patch

diff --git a/devtools/test-meson-builds.sh b/devtools/test-meson-builds.sh
index e3a5a5d51d..07d42985e6 100755
--- a/devtools/test-meson-builds.sh
+++ b/devtools/test-meson-builds.sh
@@ -117,7 +117,7 @@  export CC="clang"
 build build-arm64-host-clang $c $use_shared \
 	--cross-file $srcdir/config/arm/arm64_armv8_linux_gcc
 # all gcc/arm configurations
-for f in $srcdir/config/arm/arm*gcc ; do
+for f in $srcdir/config/arm/arm64_[bdo]*gcc ; do
 	export CC="$CCACHE gcc"
 	build build-$(basename $f | tr '_' '-' | cut -d'-' -f-2) $c \
 		$use_shared --cross-file $f