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

Petr Šabata <[email protected]> changed:

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



--- Comment #1 from Petr Šabata <[email protected]> ---
TODO: BuildRequire perl (called in SPEC)
TODO: BuildRequire perl(strict), ./Form.pm:3
TODO: BuildRequire perl(vars), ./Form.pm:4
TODO: Drop the LWP::UserAgent build-time dep, besides it being mentioned in the
Makefile.PL, it's not actually needed for build
TODO: Be modern and use DESTDIR instead of PERL_INSTALL_ROOT
TODO: Drop line 37, it's useless
TODO: Package the `ex' directory in %doc

None of those are blockers but consider fixing them before you push the
package.

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