[sagemath] 01/04: Revert "Add a temporary exception to the i386 testsuite to allow the build to pass"

[Tobias Hansen]

This is an automated email from the git hooks/post-receive script.

thansen pushed a commit to branch master
in repository sagemath.

commit 509716b134d395d855b5ea4087fecd7cc15eea64
Author: Tobias Hansen <thansen at debian.org>
Date:   Wed Jan 4 11:32:20 2017 +0100

    Revert "Add a temporary exception to the i386 testsuite to allow the build to pass"
    
    This reverts commit 7d45c6735b26b437de567c3792a59d3107321895.
---
 debian/README.Debian | 14 --------------
 debian/rules         |  7 -------
 2 files changed, 21 deletions(-)

diff --git a/debian/README.Debian b/debian/README.Debian
index 20001fe..594a821 100644
--- a/debian/README.Debian
+++ b/debian/README.Debian
@@ -42,20 +42,6 @@ At the moment, some things are not working, or working a bit roughly:
 - Jupyter notebook: there are errors in the browser console relating to loading
   MathJax fonts but this doesn't seem to interfere with the functionality.
 
-- On i386, some functions relating to brial (formerly known as polybori) will
-  cause sage to segfault, including doctests in the following files:
-
-  sage -t --long src/sage/rings/polynomial/multi_polynomial_sequence.py  # Killed due to segmentation fault
-  sage -t --long src/sage/rings/polynomial/pbori.pyx  # Killed due to segmentation fault
-  sage -t --long src/sage/crypto/mq/sr.py  # Killed due to segmentation fault
-
-  We temporarily override these failures and allow the build to pass, in order
-  to allow SageMath to pass the 2017-01-05 deadline for new packages in Debian
-  testing. This gives us about a month's breathing space to deal with this
-  properly, before the full-freeze 2017-02-05 deadline. There is a chance that
-  "deal with" might mean "drop SageMath i386 from Debian stable", so users
-  should not yet assume that sagemath:i386 will exist in the future.
-
 
 Running doctests
 ================
diff --git a/debian/rules b/debian/rules
index cb3e228..924e027 100755
--- a/debian/rules
+++ b/debian/rules
@@ -13,12 +13,6 @@ is_prerelease = false
 export SAGE_TIMEOUT_LONG = 7200
 endif
 
-ifeq (i386,$(shell dpkg-architecture -q DEB_HOST_ARCH))
-is_temporary_exception = true
-else
-is_temporary_exception = false
-endif
-
 # Use ccache if pkg.sagemath.ccache build-profile is active.
 # Your rebuilds will be *much quicker*.
 ifneq (,$(filter pkg.sagemath.ccache,$(DEB_BUILD_PROFILES)))
@@ -183,7 +177,6 @@ ifeq (,$(filter nocheck,$(DEB_BUILD_OPTIONS)))
 	$(run_tests) || true
 	# If tests fail but not by too many more, then retry them once
 	if $(is_prerelease); then :; \
-	elif $(is_temporary_exception); then :; \
 	elif debian/rules -s had-few-failures; then :; else \
 	  debian/rules -s had-not-too-many-failures && \
 	  mv sage/logs/ptestlong.log sage/logs/ptestlong.log.1 && \

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/debian-science/packages/sagemath.git