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

Add an spkg-check file for Pari #9518

Closed
sagetrac-drkirkby mannequin opened this issue Jul 16, 2010 · 7 comments
Closed

Add an spkg-check file for Pari #9518

sagetrac-drkirkby mannequin opened this issue Jul 16, 2010 · 7 comments

Comments

@sagetrac-drkirkby
Copy link
Mannequin

sagetrac-drkirkby mannequin commented Jul 16, 2010

John Cremona remarked on #9281 that Pari had a self test that could be run from make test-all, so an spkg-check file should be added to do this. I will take care of it. Hopefully John can review it.

Dave

Marked "duplicate/invalid/wontfix" because it is now in #9343. -- Jeroen Demeyer

CC: @JohnCremona @williamstein @robertwb

Component: spkg-check

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

@sagetrac-drkirkby sagetrac-drkirkby mannequin self-assigned this Jul 16, 2010
@JohnCremona
Copy link
Member

comment:1

Dave, I suggest that you leave this to be done as part of the #9343 upgrade. I have already started working on this, including reporting some issues with the testing script upstream to par-dev.

I thought you would like this plan -- and pleasantly surprised to see that after fixing bugs they add tests to the suite.

@JohnCremona JohnCremona added this to the sage-5.0 milestone Jul 16, 2010
@sagetrac-drkirkby
Copy link
Mannequin Author

sagetrac-drkirkby mannequin commented Jul 16, 2010

comment:2

Replying to @JohnCremona:

Dave, I suggest that you leave this to be done as part of the #9343 upgrade.

No problem.

I actually get some failures. I'll attach the spkg-check file as an attachment, since I'd already done it. It will save you the hassle.

I actually get some failures on my OpenSolaris machine. In fact, whilst Sage builds on OpenSolaris, it crashes at startup. So perhaps I might look at Pari as a possible source of trouble.

make[1]: Leaving directory `/export/home/drkirkby/sage-4.5/spkg/build/pari-2.3.5.p2/src/Osolaris-ix86'

real	0m21.787s
user	1m52.919s
sys	0m5.264s
Successfully installed pari-2.3.5.p2
Running the test suite.
Making test-all in Osolaris-ix86
make[1]: Entering directory `/export/home/drkirkby/sage-4.5/spkg/build/pari-2.3.5.p2/src/Osolaris-ix86'
File ../src/funclist not changed.
rm -f gp-sta
gcc  -o gp-sta -O3 -Wall -fno-strict-aliasing -fomit-frame-pointer -fPIC  -g -m64    mp.o mpinl.o Flx.o Qfb.o RgX.o alglin1.o alglin2.o arith1.o arith2.o base1.o base2.o base3.o base4.o base5.o bibli1.o bibli2.o buch1.o buch2.o buch3.o buch4.o galconj.o gen1.o gen2.o gen3.o ifactor1.o perm.o polarit1.o polarit2.o polarit3.o rootpol.o subcyclo.o subgroup.o trans1.o trans2.o trans3.o anal.o compat.o default.o errmsg.o es.o init.o intnum.o members.o sumiter.o aprcl.o elldata.o elliptic.o galois.o groupid.o kummer.o mpqs.o nffactor.o part.o stark.o subfield.o thue.o gp.o gp_init.o gp_rl.o highlvl.o whatnow.o plotport.o plotnull.o    -lm -L/export/home/drkirkby/sage-4.5/local/lib -lgmp
* Testing objets 	for gp-sta..TIME=0	for gp-dyn..TIME=0
* Testing analyz 	for gp-sta..TIME=13	for gp-dyn..TIME=15
* Testing number 	for gp-sta..TIME=13	for gp-dyn..TIME=14
* Testing polyser 	for gp-sta..TIME=3	for gp-dyn..TIME=7
* Testing linear 	for gp-sta..TIME=8	for gp-dyn..TIME=5
* Testing elliptic 	for gp-sta..TIME=9	for gp-dyn..TIME=9
* Testing sumiter 	for gp-sta..TIME=17	for gp-dyn..TIME=16
* Testing graph 	for gp-sta..TIME=4	for gp-dyn..TIME=4
* Testing program 	for gp-sta..TIME=5	for gp-dyn..TIME=8
* Testing trans 	for gp-sta..TIME=42	for gp-dyn..TIME=27
* Testing nfields 	for gp-sta..TIME=137	for gp-dyn..TIME=105
* Testing compat 	for gp-sta..TIME=157	for gp-dyn..TIME=163
* Testing ellglobalred 	for gp-sta..BUG [2]	for gp-dyn..BUG [2]
* Testing galois 	for gp-sta..TIME=7763	for gp-dyn..TIME=7455
* Testing intnum 	for gp-sta..TIME=7013	for gp-dyn..TIME=7214
* Testing qfbsolve 	for gp-sta..TIME=1465	for gp-dyn..TIME=1460
* Testing rfrac 	for gp-sta..TIME=6961	for gp-dyn..TIME=7754
* Testing round4 	for gp-sta..TIME=6535	for gp-dyn..TIME=6651
* Testing stark 	for gp-sta..TIME=16150	for gp-dyn..TIME=16396
+++ [BUG] Total bench for gp-sta is 46187
+++ [BUG] Total bench for gp-dyn is 47221

PROBLEMS WERE NOTED. The following files list them in diff format: 
Directory: /export/home/drkirkby/sage-4.5/spkg/build/pari-2.3.5.p2/src/Osolaris-ix86
	ellglobalred-sta.dif
	ellglobalred-dyn.dif
make[1]: *** [test-all] Error 1
make[1]: Leaving directory `/export/home/drkirkby/sage-4.5/spkg/build/pari-2.3.5.p2/src/Osolaris-ix86'
make: *** [test-all] Error 2
Pari failed the self-tests when running 'make test-all'
*************************************
Error testing package ** pari-2.3.5.p2 **
*************************************
sage: An error occurred while testing pari-2.3.5.p2

@sagetrac-drkirkby
Copy link
Mannequin Author

sagetrac-drkirkby mannequin commented Jul 16, 2010

spkg-check for Pari

@sagetrac-drkirkby
Copy link
Mannequin Author

sagetrac-drkirkby mannequin commented Jul 16, 2010

comment:3

Attachment: spkg-check.gz

I note there is an error in spkg-install The very first line has some unwanted characters, with what displays as "B1;2000;0c".

I leave you to sort that out!

drkirkby@hawk:~/sage-4.5/spkg/standard/pari-2.3.5.p1$ head  spkg-install
B1;2000;0c#!/bin/sh
###########################################
## PARI
###########################################

TOP=`pwd`

# As of PARI 2.3.3, the compiler flag -fPIC is not added on Solaris, but it
# needs to be if using gcc. I assume the equivalent will be needed on 
# other compilers. 
drkirkby@hawk:~/sage-4.5/spkg/standard/pari-2.3.5.p1$ 

@JohnCremona
Copy link
Member

comment:4

For an explanation of the errors see my post to pari-dev at http://pari.math.u-bordeaux.fr/archives/pari-dev-1007/threads.html and Karim's reply which I have not yet absorbed.

I have been compiling pari and running its tests for about 20 years...

@jdemeyer

This comment has been minimized.

@jdemeyer jdemeyer removed this from the sage-5.0 milestone Jul 27, 2010
@jdemeyer
Copy link

jdemeyer commented Sep 8, 2010

comment:7

There is an spkg-check in #9343 and it seems to work, so positive review.

@qed777 qed777 mannequin removed the s: positive review label Sep 10, 2010
@qed777 qed777 mannequin closed this as completed Sep 10, 2010
@qed777 qed777 mannequin added r: duplicate and removed p: major / 3 labels Sep 10, 2010
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

2 participants