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

Petr Šabata <[email protected]> changed:

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



--- Comment #5 from Petr Šabata <[email protected]> ---
(In reply to Bill Pemberton from comment #4)
> I just realized that I removed perl from the BuildRequires instead of just
> removing the version requirement.  Do I need to add that back or is it ok to
> assume the other perl BuildRequires will take care of it?

Add it back.  Although it's almost certain something will pull perl into the
buildroot, you can never know.  And since you're using it yourself, you need to
buildrequire it.

Anyhow, I trust you'll add it before pushing the package.  I ack you've fixed
everything I've pointed out so I'm going to approve this now :)

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