[Bug 1187873] Review Request: perl-Palm-PDB - Parse Palm database files

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

David Dick <ddick@xxxxxxxx> changed:

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



--- Comment #1 from David Dick <ddick@xxxxxxxx> ---
License is correct.

examples can be included in %doc

include the following BRs

perl(warnings)
perl(strict)

rpmlint is fine (ignoring incorrect spelling warnings)

package builds fine locally (koji seems to be having some SSL certificate
issues at the moment)

if you are building only for F21, 

"find $RPM_BUILD_ROOT -type f -name .packlist -exec rm -f {} \;" looks like it
can be replaced with NO_PACKLIST=1 as a Makefile.PL argument.

otherwise you'll need to adjust the %license key?

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





[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]