[Bug 175291] Review Request: perl-PatchReader - utilities to read and manipulate patches and CVS

[bugzilla at redhat.com]

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

Summary: Review Request: perl-PatchReader - utilities to read and manipulate patches and CVS


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





------- Additional Comments From stickster at gmail.com  2005-12-08 16:32 EST -------
Thanks, Ville -- some questions follow so I can learn something out of this:

(In reply to comment #1)
> - Requires: perl(:MODULE_COMPAT_*) missing

What does this mean?

> - Installs *.pm executable

You must have picked this up during the few minutes before I fixed this problem
-- the ones up there as of this morning don't have this problem, but thanks.

> - Off sync with many "best practices" from fedora-rpmdevtools' perl spec
>   template

Yeah, I should have probably started there instead of appropriating Dag's and
starting with it.

> Here's my package which should be ready to roll; "|| :" could be removed from
> %check and BuildRequires trimmed assuming recent perl packages though:
> http://cachalot.mine.nu/4/SRPMS/perl-PatchReader-0.9.5-0.2.src.rpm

Do you want me to take this one over then?

> Note: the URLs you posted apparently use some kind of a frame hack which makes
> them non-wgettable. 

Yes, but elinks should work fine if you're stuck with a text browser.

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