#14596: Make a new optional spkg for pari's databases
--------------------------------------+-------------------------------------
       Reporter:  cremona             |         Owner:  tbd         
           Type:  enhancement         |        Status:  needs_review
       Priority:  minor               |     Milestone:  sage-5.10   
      Component:  packages: optional  |    Resolution:              
       Keywords:  pari optional spkg  |   Work issues:              
Report Upstream:  N/A                 |     Reviewers:              
        Authors:  Jeroen Demeyer      |     Merged in:              
   Dependencies:                      |      Stopgaps:              
--------------------------------------+-------------------------------------

Comment (by leif):

 `spkg-check` looks a bit funny.   First, it depends on the Sage library,
 second, it uses `-tp`, and even runs the tests in verbose mode (without
 any additional message referring to the database(s) subject to
 installation).

 [[BR]]

 Since this is an optional spkg, I'd not rely on `SAGE_SRC` (and
 `SAGE_SHARE`, which you don't use [yet]) being defined, both being
 relatively new, but use `${SAGE_SRC:-${SAGE_ROOT}/devel/sage}` etc.
 instead, or do
 {{{
 #!sh
 [[ -z "$SAGE_SRC" ]] && SAGE_SRC="${SAGE_ROOT}/devel/sage"

 [[ -z "$SAGE_SHARE" ]] && SAGE_SHARE="${SAGE_LOCAL}/share"
 }}}

 (Then I'd use `"${SAGE_SHARE}/pari/" as the destination directory.)

 I'd also rather use `gp` in `spkg-check`, or do some simpler sanity check
 on the installed files.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/14596#comment:12>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica, 
and MATLAB

-- 
You received this message because you are subscribed to the Google Groups 
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to