Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

developer's guide: document the variable SAGE_CHECK #8262

Closed
sagetrac-mvngu mannequin opened this issue Feb 14, 2010 · 7 comments
Closed

developer's guide: document the variable SAGE_CHECK #8262

sagetrac-mvngu mannequin opened this issue Feb 14, 2010 · 7 comments

Comments

@sagetrac-mvngu
Copy link
Mannequin

sagetrac-mvngu mannequin commented Feb 14, 2010

As the subject says.

Component: documentation

Issue created by migration from https://trac.sagemath.org/ticket/8262

@sagetrac-mvngu sagetrac-mvngu mannequin added this to the sage-4.5 milestone Feb 14, 2010
@sagetrac-mvngu sagetrac-mvngu mannequin self-assigned this Feb 14, 2010
@sagetrac-drkirkby
Copy link
Mannequin

sagetrac-drkirkby mannequin commented Feb 14, 2010

comment:1

See also #8263, which aims to document all environment variables. SAGE_CHECK is not the only one which is either undocumented, or poorly documented.

@qed777
Copy link
Mannequin

qed777 mannequin commented Mar 2, 2010

comment:2

Some somewhat related data: SAGE_LOCAL/bin/sage-spkg contains

    cd $BASEDIR
    if [ "$SAGE_CHECK" != "" -a -f spkg-check ]; then
        echo "Running the test suite."
        chmod +x spkg-check
        ./spkg-check
        if [ $? -ne 0 ]; then

but

$ \ls -1 /home/release/latest/sage-4.3.3/spkg/standard/*.spkg | awk '{print "tar jxvf "$0}' > zz
$ . zz
$ gr -A3 -B2 spkg-check */spkg-install
flint-1.5.0.p3/spkg-install-
flint-1.5.0.p3/spkg-install-if [ "$SAGE_CHECK" = "yes" ]; then
flint-1.5.0.p3/spkg-install:    cd ..; ./spkg-check
flint-1.5.0.p3/spkg-install-fi
--
mpfr-2.4.1.p1/spkg-install-# Do not bypass the checks, as some MPFR failures
mpfr-2.4.1.p1/spkg-install-# have been observed, so MPFR should be carefully tested.
mpfr-2.4.1.p1/spkg-install:cd ..; ./spkg-check
--
mpir-1.2.2.p0/spkg-install-
mpir-1.2.2.p0/spkg-install-if [ "$SAGE_CHECK" = "yes" ]; then
mpir-1.2.2.p0/spkg-install:    cd ..; ./spkg-check
mpir-1.2.2.p0/spkg-install-fi

In particular, with SAGE_CHECK="yes", the flint's long-running tests are run twice.

@rlmill
Copy link
Mannequin

rlmill mannequin commented Jul 16, 2010

comment:3

Can this ticket be closed now?

@sagetrac-mvngu
Copy link
Mannequin Author

sagetrac-mvngu mannequin commented Jul 16, 2010

comment:4

Replying to @rlmill:

Can this ticket be closed now?

Yes, this ticket can be closed. The issue of this ticket has been fixed in #8263. I leave it to the release manager to close the current ticket.

@rlmill
Copy link
Mannequin

rlmill mannequin commented Jul 16, 2010

comment:5

Right. Just wanted a second opinion. Thanks!

@rlmill rlmill mannequin added r: duplicate and removed p: major / 3 labels Jul 16, 2010
@rlmill rlmill mannequin closed this as completed Jul 16, 2010
@rlmill rlmill mannequin removed this from the sage-4.5 milestone Jul 16, 2010
@qed777
Copy link
Mannequin

qed777 mannequin commented Jul 16, 2010

comment:6

Revisiting the comment above for Sage 4.5, I get

$ \ls -1 /path/to/sage-4.5/spkg/standard/*.spkg | awk '{print "tar jxvf "$0}' > unpackem
$ . unpackem
$ egrep -A3 -B2 -i SAGE_CHECK\|spkg-check */spkg-install
cliquer-1.2.p5/spkg-install-fi
cliquer-1.2.p5/spkg-install-
cliquer-1.2.p5/spkg-install:if [ "$SAGE_CHECK" = "yes" ]; then
cliquer-1.2.p5/spkg-install-    echo "Compiling and running the test cases of cliquer..."
cliquer-1.2.p5/spkg-install-
cliquer-1.2.p5/spkg-install-    make testcases
--
mpfr-2.4.2/spkg-install-# Do not bypass the checks, as some MPFR failures
mpfr-2.4.2/spkg-install-# have been observed, so MPFR should be carefully tested.
mpfr-2.4.2/spkg-install:cd ..; ./spkg-check
--
mpir-1.2.2.p1/spkg-install-fi
mpir-1.2.2.p1/spkg-install-
mpir-1.2.2.p1/spkg-install:if [ "$SAGE_CHECK" = "yes" ]; then
mpir-1.2.2.p1/spkg-install:    cd ..; ./spkg-check
mpir-1.2.2.p1/spkg-install-fi

If no one objects, I can open tickets for MPIR and Cliquer.

@qed777
Copy link
Mannequin

qed777 mannequin commented Jul 17, 2010

comment:7

Replying to @qed777:

If no one objects, I can open tickets for MPIR and Cliquer.

These are now #9522 (MPIR) and #9521 (Cliquer).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

0 participants