#6612: [with patch, needs review] sage-update selects a suiteable mirror
--------------------------+-------------------------------------------------
Reporter: schilly | Owner: tbd
Type: enhancement | Status: new
Priority: major | Milestone: sage-4.1.2
Component: distribution | Keywords:
Reviewer: | Author:
Merged: |
--------------------------+-------------------------------------------------
Comment(by mvngu):
The two patches `6612-sage-update-mirror-network-r1.patch` and `6612-sage-
update-mirror-network-r2.patch` look good to me. I have attached a
reviewer patch `trac_6612-reviewer.patch` that include the following
changes:
* Some formatting fixes to `sage-sage`.
* Remove the re-import of `urllib` and `socket`.
* Remove the unused import of `subprocess`.
* Remove multiple imports on one line.
* Use three double quotation marks `"` instead of three single quotation
marks `'`.
* Use 4 space indentation.
* Spell checking.
If my patch gets some thumbs up, then the whole ticket can be merged for
Sage 4.1.2.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/6612#comment:14>
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
-~----------~----~----~----~------~----~------~--~---
