[Bug 522777] Review Request: perl-Verilog-Perl - 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=522777





--- Comment #3 from Chitlesh GOORAH <chitlesh@xxxxxxxxx>  2009-09-24 12:21:33 EDT ---
(In reply to comment #2)
> So this is just a rename review request?  I'll do a quick runthrough.

Yes it is just a rename review request.

> 
> The Obsoletes: and Provides: 

done

> 
> There are a couple of provides which look a bit odd.  I think  that for some
> bizarre reason the automatic dependency finder actually parses "package"
> statements in some verilog source (installed as documentation) as perl
> "package" statements, and so you end up with the bogus "perl(imp_test_pkg)" and
> "perl(mypackage)".  That's pretty bad behavior from rpmbuild almost certainly
> worthy of a bug, but you'll still need to filter those dependencies. 
> Unfortunately you can't use the general dependency filtering mechanism just
> introduced because this package is arch-specific.  Perhaps the mechanism at
> http://fedoraproject.org/wiki/Packaging:Perl#Filtering_Requires:_and_Provides
> will work for you.

done

> 
> The package review process needs reviewers!  If you haven't done any package
> reviews recently, please consider doing one.

I'm doing my package reviews regularly :)

Spec URL: http://chitlesh.fedorapeople.org/RPMS/perl-Verilog-Perl.spec
SRPM URL:
http://chitlesh.fedorapeople.org/RPMS/perl-Verilog-Perl-3.213-2.fc11.src.rpm

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