#12530: Improve the sage-combinat script to support guards for developpers
versions
of Sage
------------------------------+---------------------------------------------
Reporter: nthiery | Owner: leif
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-5.0
Component: scripts | Keywords:
Work_issues: | Upstream: N/A
Reviewer: Florent Hivert | Author: Nicolas M. Thiéry
Merged: | Dependencies:
------------------------------+---------------------------------------------
Comment(by nthiery):
Replying to [comment:7 hivert]:
> Replying to [comment:6 nthiery]:
> > Oops, the guard should not have matched the regexp in the first place;
I had forgotten the "$" at the end of it for an exact match. The updated
patch still complains as you suggest, since this is likely to be a
typicall typo.
>
> Should this be a warning or an error ? I think I'd rather have an error
than a warning. The reason is that the warning is issued in the middle of
a lot of info message and is likely to go unnoticed. Why do you prefer
warning ?
I pondered about this too. My rationale was the following: imagine a
beginner installs the queue and runs into this issue; if the queue is
really screwed up, their will be an error later one anyway. Otherwise, he
could be lucky and have the queue apply reasonnably and be able to get to
work; so why blocking him.
That being said, it is indeed buried in the middle of a bunch of info,
though quite visible for someone scrolling up.
So I don't know. Maybe that would be a use case for throwing an error by
default, and a warning under -f. What do you think?
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12530#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.
