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

Petr Viktorin <pvikt...@redhat.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |pvikt...@redhat.com
         Whiteboard|                            |NotReady



--- Comment #1 from Petr Viktorin <pvikt...@redhat.com> ---
The point of this library is that it does not require python-configobj,
python-xmltodict, and PyYAML. Could you remove them? (From Requires, not
BuildRequires -- they're still needed for tests.)
The package does need python-six, can you add that to Requires & BuildRequires?

The Source0 URL doesn't work.

A mostly aesthetic nitpick: please trim the changelog since this is a new
package.

Otherwise the package looks good!

-- 
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
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to