https://bugzilla.redhat.com/show_bug.cgi?id=1206743

Ralph Bean <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |[email protected]
              Flags|                            |fedora-review+



--- Comment #2 from Ralph Bean <[email protected]> ---
Vic.  Thanks for taking a look at it.  If you're interested in someday becoming
an official reviewer, it can help to post some more details about what you
looked at.

Garrett, It would be nice if:

1) you could file a ticket with upstream asking them to include the license
full text, and link to that ticket from the spec file.
2) you were more specific about what you are packaging in site-packages.  it is
easy to make mistakes later down the road when you package '*'.


Otherwise, it looks good.  (license checks out.  no bundling).

Package is approved.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to