PR#37: Add %ctest timeout on riscv64 - rpms/cmake

rpms / cmake

#37 Add %ctest timeout on riscv64

Merged 4 months ago by orion. Opened 4 months ago by rjones.

rpms/ rjones/cmake riscv64-timeout into rawhide

Add %ctest timeout on riscv64

Richard W.M. Jones • 4 months ago

8d718fa

macros.cmake.in

file modified

+7 -1

		`@@ -44,7 +44,13 @@`
		`DESTDIR="%{buildroot}" %__cmake --install "%{__cmake_builddir}"`

		`%ctest(:-:h:j:u:v:A:C:D:E:F:H:I:L:M:N:O:Q:R:S:T:U:V:) \`
		`- %__ctest --test-dir "%{__cmake_builddir}" --output-on-failure --force-new-ctest-process %{?_smp_mflags} %{**}`
		`+ %__ctest --test-dir "%{__cmake_builddir}" \\\`
		`+ --output-on-failure \\\`
		`+ --force-new-ctest-process \\\`
		`+ %ifarch riscv64 \`
		`+ --timeout 6000 \\\`
		`+ %endif \`
		`+ %{?_smp_mflags} %{**}`


		`%cmake@@CMAKE_MAJOR_VERSION@@ %cmake`

rjones commented 4 months ago

Since our riscv64 builders are slow at the moment some tests fail with
timeouts, eg:
https://src.fedoraproject.org/rpms/kokkos/pull-request/11

Adding the --timeout flag on riscv64 only increases the timeout and
makes the tests pass. Adding this centrally makes it easier to remove
it later once faster hardware becomes widely available.

Signed-off-by: Richard W.M. Jones rjones@redhat.com

rjones commented 4 months ago

(Adding @davidlt @tekkamanninja )