#8654: add "sage -sqlite3" command line option
---------------------------------------------+------------------------------
Reporter: was | Owner: jason
Type: enhancement | Status: needs_work
Priority: minor | Milestone: sage-4.7.2
Component: misc | Keywords:
Work_issues: | Upstream: N/A
Reviewer: William Stein, Leif Leonhardy | Author: R. Andrew
Ohana, Jeroen Demeyer
Merged: | Dependencies:
---------------------------------------------+------------------------------
Changes (by leif):
* status: needs_review => needs_work
* reviewer: William Stein => William Stein, Leif Leonhardy
Comment:
Testing for the exact version number in `sage/tests/cmdline.py` is
certainly suboptimal.
`install_scripts()` (in `sage/misc/dist.py`) needs to be updated, too.
----
Could do some bikeshedding here:
* The following
{{{
#!sh
some_program "$@"
exit $?
}}}
should be just
{{{
#!sh
exec some_program "$@"
}}}
(perhaps followed by `exit $?` if we want to go triple-safe).
* The help messages should use the long option form (`--foo` rather than
`-foo`). [[BR]]
Stating that `sage -foobar` runs Sage's version of `foobar` foreach
foobar \in {A,...,Z} is of questionable value.
* Doing
{{{
#!sh
if [ foo = bar -o foo = BAR ]; then
...
fi
if [ foo = baz -o foo = BAZ ]; then
...
fi
if [ foo = whatever ]; then
...
fi
}}}
is very poor. It should simply be
{{{
#!sh
case foo in
bar|BAR)
...
;;
baz|BAZ)
...
;;
...
esac
}}}
which is faster, safer, easier to read and also less error-prone.
One could change ''all'' of these at once on a separate ticket, but it's
IMHO dumb to always replicate the same bad code again. So I'd change it
at least in the new parts (and in general those one touches anyway or the
ones near to them.)
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/8654#comment:10>
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.
