[Bug 476386] Review Request: perl-verilog - Verilog parsing routines

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

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #3 from manuel wolfshant <wolfy@xxxxxxxxxxxxxxxxxx>  2008-12-15 02:51:21 EDT ---
Are you targeting only F10 ? Because the src.rpm from #2 still does not build
in rawhide: http://koji.fedoraproject.org/koji/taskinfo?taskID=998879

Note that according to
http://koji.fedoraproject.org/koji/getfile?taskID=997866&name=build.log, "'CP'
is not a known MakeMaker parameter name." and therefore your little trick to
preserve timestamps when using cp is ignored.

http://koji.fedoraproject.org/koji/getfile?taskID=997866&name=perl-verilog-3.044-2.fc10.x86_64.rpm
gives no less than 53 warnings in rpmlint, all of them being either
"spurious-executable-perm" or "doc-file-dependency". Could you please fix that
? I do not think that bringing in perl(Test) or perl(Data::Dumper) is a good
idea: people wanting to run the tests could install the dependencies manually.

A more delicate problem are the scripts named vpm and vppp. According to
https://fedoraproject.org/wiki/PackageMaintainers/Packaging_Tricks#Use_of_common_namespace
, commands whose names are shorter than 5 letters should be avoided.
Fortunately vpm does not seem to conflict with anything, but vppp is also the
name of the Virtual PPP project by packagestorm and is also used by
http://www.linux-kheops.com/pub/vppp/vpppGB.html

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

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