#12608: Remove "Dependencies check" from ppl
---------------------------+------------------------------------------------
Reporter: jdemeyer | Owner: tbd
Type: enhancement | Status: new
Priority: major | Milestone: sage-5.0
Component: packages | Keywords:
Work_issues: | Upstream: N/A
Reviewer: | Author:
Merged: | Dependencies:
---------------------------+------------------------------------------------
Description changed by jdemeyer:
Old description:
> Remove the following block from the PPL spkg:
> {{{
> # Dependencies check
> MPIR_VERSION=`cd $SAGE_ROOT/spkg/; ./standard/newest_version mpir`
> if [ $? -ne 0 ]; then
> echo "Failed to find mpir. Please install the mpir spkg"
> exit 1
> fi
> }}}
>
> Although suggested by the developer manual, it doesn't do what is
> intented (it only checks that the mpir spkg exists, not that it has been
> installed). It's also not needed, since it's a standard package
> depending on a standard package.
New description:
Remove the following block from the PPL spkg:
{{{
# Dependencies check
MPIR_VERSION=`cd $SAGE_ROOT/spkg/; ./standard/newest_version mpir`
if [ $? -ne 0 ]; then
echo "Failed to find mpir. Please install the mpir spkg"
exit 1
fi
}}}
Although suggested by the developer manual, it doesn't do what is intented
(it only checks that the mpir spkg exists, not that it has been
installed). It's also not needed, since it's a standard package depending
on a standard package.
While I was changing the package anyway: use `patch` for patching.
--
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12608#comment:1>
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.
