#12609: Update developer guide for SPKGs
------------------------------+---------------------------------------------
Reporter: jdemeyer | Owner: mvngu
Type: defect | Status: needs_review
Priority: minor | Milestone: sage-5.0
Component: documentation | Keywords:
Work_issues: | Upstream: N/A
Reviewer: Jeroen Demeyer | Author: John Palmieri, Jeroen Demeyer
Merged: | Dependencies:
------------------------------+---------------------------------------------
Changes (by newvalueoldvalue):
* reviewer: => Jeroen Demeyer
* author: John Palmieri => John Palmieri, Jeroen Demeyer
Old description:
> This ticket is about updating the developer guide for #10492.
>
> The developer guide suggests
> (http://sagemath.org/doc/developer/producing_spkgs.html#the-file-spkg-
> txt)
> {{{
> If your package depends on another package, say boehmgc, then you
> should check that this other package has been installed. Your spkg-
> install script should check that it exists, with code like the following:
>
> BOEHM_GC=`cd $SAGE_ROOT/spkg/standard/; ./newest_version boehm_gc`
> if [ $? -ne 0 ]; then
> echo "Failed to find boehm_gc. Please install the boehm_gc
> spkg"
> exit 1
> fi
> }}}
> This is totally wrong. It only checks that a spkg ''exists'', not that
> it has been installed.
>
> See #12608 for removing this wrong checking from PPL.
>
> Apply [attachment:trac_12609-doc-developer.patch] to the main Sage
> library.
New description:
This ticket is about updating the developer guide for #10492.
The developer guide suggests
(http://sagemath.org/doc/developer/producing_spkgs.html#the-file-spkg-txt)
{{{
If your package depends on another package, say boehmgc, then you
should check that this other package has been installed. Your spkg-install
script should check that it exists, with code like the following:
BOEHM_GC=`cd $SAGE_ROOT/spkg/standard/; ./newest_version boehm_gc`
if [ $? -ne 0 ]; then
echo "Failed to find boehm_gc. Please install the boehm_gc
spkg"
exit 1
fi
}}}
This is totally wrong. It only checks that a spkg ''exists'', not that it
has been installed. See #12608 for removing this wrong checking from PPL.
The patches below also include various fixes to "Producing New Sage
Packages".
Apply [attachment:trac_12609-doc-developer.patch] and [12609_review.patch]
to the main Sage library.
--
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12609#comment:7>
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 post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
