#11991: record time, version in sage-starts
------------------------------+---------------------------------------------
Reporter: jhpalmieri | Owner: leif
Type: enhancement | Status: positive_review
Priority: minor | Milestone: sage-4.8
Component: scripts | Keywords:
Work_issues: | Upstream: N/A
Reviewer: Leif Leonhardy | Author: John Palmieri
Merged: | Dependencies: #11926
------------------------------+---------------------------------------------
Changes (by leif):
* status: needs_review => positive_review
* dependencies: => #11926
Comment:
Actually only depends on ''one part'' (the one to `sage-starts`) ''of one
patch'' ([attachment:ticket:11926:11926_sage_starts.patch]) at #11926.
`sage-starts` could by the way be more robust, i.e., it shouldn't print
the following if the current working directory just happens to not be
`$SAGE_ROOT`, and `SAGE_ROOT` isn't set.
{{{
#!sh
$ bin/sage-starts
Testing that Sage starts...
[2011-11-05 23:15:43]
bin/sage-starts: line 14: spkg/pipestatus: No such file or directory
Sage failed to start up.
Please email sage-devel (http://groups.google.com/group/sage-devel)
explaining the problem and send the log file
/home/leif/Sage/sage-4.7.2-gcc-4.5.1/local/start.log
Describe your computer, operating system, etc.
}}}
(Also note the logfile's name / location. The log just contains the time
stamp, and left-over files are likely to pollute repositories; only the
root repository ignores `*.log` files. I'm not saying that this is
something a user will typically run into [unless e.g. he/she includes
`$SAGE_LOCAL/bin` into his/her `PATH`, which apparently some do, although
they clearly shouldn't^*^], but one shouldn't print a message that
encourages people to submit useless error reports.)
[[BR]]
Otherwise '''positive review''', since that's not really related to the
purpose of this ticket. Feel free to fix it here (or elsewhere) though.
----
^*^ We may add this to the Installation Guide (that one shouldn't put
`$SAGE_ROOT/local/bin/` into `PATH`), but IMHO most scripts there should
do the "usual" sanity check, i.e. for example test whether `SAGE_LOCAL` is
defined, since most (or at least many) depend on the Sage environment
being already set up.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11991#comment:11>
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.
