[Bug 639684] Review Request: perl-Pegex - Pegex Parser Generator

bugzilla at redhat.com bugzilla at redhat.com
Wed Oct 27 14:08:55 UTC 2010


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

--- Comment #1 from Marcela Mašláňová <mmaslano at redhat.com> 2010-10-27 10:08:54 EDT ---
- package meets naming guidelines OK
- package meets packaging guidelines OK
- license (GPL+ or Artistic) OK, text in %doc, matches source OK
- spec file legible, in am. english OK
- source matches upstream OK
- package compiles on devel (x86) OK
- no missing BR OK
- no unnecessary BR !OK
- no locales OK
- not relocatable OK
- owns all directories that it creates OK
- no duplicate files OK
- permissions ok OK
- %clean ok OK
- macro use consistent OK
- code, not content OK
- no need for -docs OK
- nothing in %doc affects runtime OK
- no need for .desktop file  OK

Not needed:
BuildRequires:  perl >= 1:5.8.3
BuildRoot:      %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
^ you don't need specify Buildroot for F-13 and higher and EL-6

perl-Pegex.src: W: invalid-url Source0:
http://search.cpan.org/CPAN/authors/id/I/IN/INGY/Pegex-0.10.tar.gz <urlopen
error timed out>
Not sure what's wrong, this link worked for me.

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


More information about the package-review mailing list