#10303: clean up sage-check-64 and use of SAGE64
--------------------------------------------+-------------------------------
Reporter: jhpalmieri | Owner: GeorgSWeber
Type: defect | Status: positive_review
Priority: minor | Milestone: sage-4.6.1
Component: build | Keywords: 64
Author: John Palmieri | Upstream: N/A
Reviewer: David Kirkby, Leif Leonhardy | Merged:
Work_issues: |
--------------------------------------------+-------------------------------
Comment(by leif):
Replying to [comment:50 jhpalmieri]:
> Leif, the same goes for you: if you see a problem like missing line
breaks in the commit message, and if it seems clear that other work on the
ticket has stopped, why not just fix it? That seems more productive than
waiting until someone else (like the release manager) notices it and then
commenting "I was aware of that."
As I said, I considered it a minor issue (and was pretty sure ''you'' were
also aware of it), so did not even mention it.
Perhaps I should have, but it's IMHO relatively useless to upload another
patch just fixing the commit message, since that usually causes more
confusion. (I actually did so on other tickets, because ''I knew'' the
author was busy / unavailable.)
If we had different access rights on trac (e.g. reviewers as "invited" co-
authors with the ability to change attachments of others), that would be
different.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/10303#comment:51>
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.
