[Bug 464117] Review Request: perl-SVN-Notify - Perl module for Subversion activity notification

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


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





--- Comment #10 from Timon <timosha at gmail.com>  2009-06-19 09:43:01 EDT ---
>Requires:       perl(:MODULE_COMPAT_%(eval "`%{__perl} -V:version`"; echo $version))
http://fedoraproject.org/wiki/Packaging:Perl
All perl modules must include the versioned MODULE_COMPAT Requires:
Requires:  perl(:MODULE_COMPAT_%(eval "`%{__perl} -V:version`"; echo $version))
This is to ensure that perl packages have a dependency on a perl which provides
the appropriate versioned directory structure (otherwise, the modules won't be
found). 

>Requires:       perl-SVN-Notify = %{version}-%{release}
fixed

> We should remove the following lines:
>find $RPM_BUILD_ROOT -type d -depth -exec rmdir {} 2>/dev/null \;
>chmod -R u+rwX,go+rX,go-w $RPM_BUILD_ROOT/*
fixed

>The descriptions for the svnnotify contains a line with more the 79 characters.
fixed

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