#11021: clean up sage-spkg
--------------------------+-------------------------------------------------
Reporter: jhpalmieri | Owner: tbd
Type: defect | Status: needs_review
Priority: minor | Milestone: sage-4.7.1
Component: packages | Keywords:
Work_issues: | Upstream: N/A
Reviewer: Kelvin Li | Author: Leif Leonhardy
Merged: | Dependencies:
--------------------------+-------------------------------------------------
Comment(by ltw):
Replying to [comment:6 leif]:
> Ok, though I wouldn't have changed the "error" message; the previous one
sounds a bit friendlier. (Actually a non-zero status code can have a
couple of reasons there, but I don't mind. ;-) )
A few lines above, there is an error message {{{"Package $PKG_NAME not
found"}}}. So I thought {{{"File SPKG.txt not found in package
$PKG_NAME"}}} would follow the same grammar. But I can change it back, no
problem. :-)
> (We could e.g. pipe the `tar` output to some sort of `sed -e '/==
Changelog ==/,$d'` and perhaps give some message indicating it was
omitted.
That's pretty hackish in my opinion. It would impose an undocumented
constraint on the format of SPKG.txt, which does not have any formatting
validation otherwise. I'm not sure that truncating SPKG.txt is even
desirable.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11021#comment:8>
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.
