Skip to content
This repository was archived by the owner on Feb 1, 2023. It is now read-only.

Commit ca36a87

Browse files
author
Matthias Koeppe
committed
build/pkgs/*/spkg-check.in: Do not print misleading error messages: avoid sdh_make
1 parent 9b213a4 commit ca36a87

File tree

11 files changed

+22
-15
lines changed

11 files changed

+22
-15
lines changed

build/bin/sage-dist-helpers

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -184,6 +184,11 @@ sdh_make() {
184184
}
185185

186186

187+
sdh_make_check() {
188+
echo "Checking $PKG_NAME"
189+
${MAKE:-make} check "$@" || sdh_die "Failures checking $PKG_NAME"
190+
}
191+
187192
sdh_make_install() {
188193
echo "Installing $PKG_NAME"
189194
if [ -n "$SAGE_DESTDIR" ]; then

build/pkgs/awali/spkg-check.in

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,2 @@
11
cd src/_build
2-
sdh_make check
2+
sdh_make_check

build/pkgs/barvinok/spkg-check.in

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,2 @@
11
cd src
2-
sdh_make check
2+
sdh_make_check

build/pkgs/cocoalib/spkg-check.in

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,2 @@
11
cd src/
2-
sdh_make check
2+
sdh_make_check

build/pkgs/e_antic/spkg-check.in

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,4 +5,4 @@
55
###############################################################################
66

77
cd src
8-
sdh_make check
8+
sdh_make_check
Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,9 @@
11
cd src
2-
sdh_make check
2+
# Do not use sdh_make or sdh_make_check here because we are doing our own error
3+
# handling below
4+
${MAKE:-make} check
35

46
if [ $? -ne 0 ]; then
5-
cat tests/test-suite.log
6-
exit 1
7+
cat tests/test-suite.log
8+
exit 1
79
fi

build/pkgs/isl/spkg-check.in

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,2 @@
11
cd src
2-
sdh_make check
2+
sdh_make_check

build/pkgs/openblas/spkg-check.in

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -17,18 +17,18 @@ if [ "x$SAGE_FAT_BINARY" = "xyes" ]; then
1717
OPENBLAS_CONFIGURE="$OPENBLAS_CONFIGURE TARGET=PRESCOTT"
1818
fi
1919

20-
sdh_make tests $OPENBLAS_CONFIGURE
20+
${MAKE:-make} tests $OPENBLAS_CONFIGURE
2121
if [ $? -ne 0 ]; then
2222
# First make sure we already didn't set a target
2323
if [[ $OPENBLAS_CONFIGURE == *"TARGET"* ]]; then
24-
sdh_die "Error while running the OpenBlas testsuite ... exiting"
24+
sdh_die "Error while running the OpenBLAS testsuite ... exiting"
2525
else
2626
# The recommended TARGET is ATOM if CPU fails
2727
# See https://github.com/xianyi/OpenBLAS/issues/1204
2828
OPENBLAS_CONFIGURE="$OPENBLAS_CONFIGURE TARGET=ATOM"
2929
echo "Error while testing the OpenBLAS testsuite"
30-
echo "Retrying the OpenBAS testsuit with TARGET=ATOM"
31-
sdh_make tests $OPENBLAS_CONFIGURE
30+
echo "Retrying the OpenBLAS testsuite with TARGET=ATOM"
31+
${MAKE:-make} tests $OPENBLAS_CONFIGURE
3232
if [ $? -ne 0 ]; then
3333
sdh_die "Error while running the OpenBLAS testsuite ... exiting"
3434
fi

build/pkgs/polylib/spkg-check.in

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,2 @@
11
cd src
2-
sdh_make check
2+
sdh_make_check

build/pkgs/polymake/spkg-check.in

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,2 @@
11
cd src/
2-
sdh_make test
2+
${MAKE:-make} test || sdh_die "Failures checking polymake"

0 commit comments

Comments
 (0)