#16187: Show (all) installed versions of optional/experimental packages in sage-
list-packages
-------------------------------------+-------------------------------------
       Reporter:  vbraun             |        Owner:
           Type:  defect             |       Status:  positive_review
       Priority:  major              |    Milestone:  sage-6.2
      Component:  build              |   Resolution:
       Keywords:  --optional         |    Merged in:
  --experimental SAGE_SERVER         |    Reviewers:  Leif Leonhardy
        Authors:  Volker Braun       |  Work issues:
Report Upstream:  N/A                |       Commit:
         Branch:                     |  49b8b4ced71fd942193f22281255aeefdb7b0eb3
  u/leif/ticket/16187                |     Stopgaps:
   Dependencies:                     |
-------------------------------------+-------------------------------------

Comment (by vdelecroix):

 Hello,

 It is much better now! Some comments about the output (I did not read
 carefully the code for getting the values).

 - Trailing whitespaces line 19 and 76.
 - I would prefer use .format as it is cleaner, i.e. replace line 76 with
    {{{
    print "{:.<40}{}".format(fullname, status)
    }}}
 - To be Python 3 compatible, it would be nice to replace {{{print X}}}
 with {{{print(X)}}} and {{{raise Error, "Message"}}} with {{{raise
 Error("Message")}}}.

--
Ticket URL: <http://trac.sagemath.org/ticket/16187#comment:16>
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 unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.

Reply via email to