#16759: install_package() is obsolete
-------------------------------------+-------------------------------------
Reporter: jdemeyer | Owner:
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-6.9
Component: misc | Resolution:
Keywords: | Merged in:
Authors: Jeroen Demeyer | Reviewers:
Report Upstream: N/A | Work issues:
Branch: | Commit:
u/jdemeyer/install_package___is_obsolete|
ec6c23a895a72e46b4920ee08a5de73a3d269791
Dependencies: | Stopgaps: #16760
-------------------------------------+-------------------------------------
Comment (by jhpalmieri):
Channeling Nathann: regarding changes like `sage -i cryptominisat &&
make`, do we need to say something about being in `SAGE_ROOT` first?
This is no longer a stopgap, so in `misc.package.install_package`, should
we be returning some other error message? Maybe `NotImplementedError`? I
agree that `install_package('blah')` won't work much of the time for the
reasons you give, so a deprecation isn't the right course of action, but a
stopgap implies that the situation is temporary and is going to be fixed.
This situation may be temporary, but only in the sense that
`install_package()` may be completely deleted eventually. Also, with a
stopgap, you only get the message the first time you run it. If we raise
an error, you will get it every time, which might be better.
I'm not sure if the changes to `trac.py` belong here or on a separate
ticket. I don't care that much, though.
Also, please make the following change (or something similar):
{{{
#!diff
diff --git a/src/sage/misc/package.py b/src/sage/misc/package.py
index de4d206..9d727d9 100644
--- a/src/sage/misc/package.py
+++ b/src/sage/misc/package.py
@@ -19,7 +19,7 @@ components:
Use the :func:``optional_packages`` command to list all
optional packages available on the central Sage server.
-Actually installing the packages should be done via the Sage command
+Actually installing the packages should be done via the command
line, using the following commands:
- ``sage -i PACKAGE_NAME`` -- install the given package
}}}
Also, the sentences at the top of that file describing Sage packages are
outdated and should probably just be replaced with a pointer to the
Developer's Guide. If you are willing to do that, too, that would be nice.
--
Ticket URL: <http://trac.sagemath.org/ticket/16759#comment:28>
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.
For more options, visit https://groups.google.com/d/optout.
