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

Petr Šabata <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|fedora-review?              |fedora-review+



--- Comment #15 from Petr Šabata <[email protected]> ---
(In reply to Christopher Meng from comment #14)
> It seems that BR some modules in perl package itself is needed now, right?

Indeed, there's basically no difference between them.

I've gone through the new version code and I'm going to approve this now but:
- Some of the explicit Requires can be dropped as rpmbuild will detect them
automagically (Data::MessagePack, parent, Mouse)
- You can add BR perl(B) to be complete

You can do those changes after the import.

-- 
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