Skip to content

Commit

Permalink
sagemathgh-37764: ECM-related tests fail after an incremental build
Browse files Browse the repository at this point in the history
### TL;DR:

This is because setting the default ECMBIN=ecm is only done when using
the configure test, but not when skipping the configure test because the
spkg is already installed.

### Steps to reproduce:
* ecm is not installed in the base os
* Sage make distclean && make succeeeds
* Tests succeed
* re-running ./bootstrap && ./configure && make works
* Tests now fail with PermissionError: [Errno 13] Permission denied: ''

### Dependencies
See also:
 * sagemath#37701
 * sagemath#37011 (comment)

URL: sagemath#37764
Reported by: Volker Braun
Reviewer(s): Matthias Köppe
  • Loading branch information
Release Manager committed Apr 8, 2024
2 parents 551139c + c8d260a commit 6d187b1
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 7 deletions.
6 changes: 3 additions & 3 deletions build/pkgs/configure/checksums.ini
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
tarball=configure-VERSION.tar.gz
sha1=8cfc13031178334306e8a2745ae9d822790f8029
md5=25296e38dfa8eaf36644f8e6ff038c9e
cksum=991738162
sha1=0223519e28f35b4163a994a4fd6bbed3fdbffdbe
md5=53f144a31102971db15954c7229d252e
cksum=4202935372
2 changes: 1 addition & 1 deletion build/pkgs/configure/package-version.txt
Original file line number Diff line number Diff line change
@@ -1 +1 @@
850ed07c17579aa4c092159ca7042545ba6b3213
8b9a3b1f938364d711320a4e017cdf50dd342eb7
9 changes: 6 additions & 3 deletions build/pkgs/ecm/spkg-configure.m4
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
SAGE_SPKG_CONFIGURE([ecm], [
SAGE_SPKG_CONFIGURE([ecm], [dnl CHECK - test whether the package is already installed
m4_pushdef([SAGE_ECM_MINVER],[7.0.4])
ECMBIN=ecm
SAGE_SPKG_DEPCHECK([gmp], [
AC_CHECK_HEADER(ecm.h, [
AX_ABSOLUTE_HEADER([ecm.h])
Expand Down Expand Up @@ -35,5 +34,9 @@ SAGE_SPKG_CONFIGURE([ecm], [
], [sage_spkg_install_ecm=yes])
])
m4_popdef([SAGE_ECM_MINVER])
AC_SUBST(SAGE_ECMBIN, $ECMBIN)
], [dnl REQUIRED-check is empty
], [dnl PRE - always perform
ECMBIN=ecm
], [dnl POST - always perform
AC_SUBST([SAGE_ECMBIN], ["$ECMBIN"])
])

0 comments on commit 6d187b1

Please sign in to comment.