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-Font-TTF (part of the dejavu-fonts toolchain)
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=180034
------- Additional Comments From Nicolas.Mailhot(a)laPoste.net 2006-02-05 05:42 EST -------
(In reply to comment #3)
> Nicolas, would you please provide an SRPM?
Since the SRPM is only 148 kiB I'll attach it too
--
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.
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-Font-TTF (part of the dejavu-fonts toolchain)
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=180034
------- Additional Comments From Nicolas.Mailhot(a)laPoste.net 2006-02-05 05:33 EST -------
(In reply to comment #2)
> Generally you should supply a src.rpm; it makes the review process a bit easier
> and some of the review items require having it.
I know, it's just that my isp changed it's upload rules and I couldn't locate
the new ones yesterday
> My rpmlint complains:
> W: perl-Font-TTF wrong-file-end-of-line-encoding
> /usr/share/doc/perl-Font-TTF-0.37/README.TXT
>
> and indeed README.TXT has CRLF endings. However, there are several packages
> which include documentation files with DOS-style line endings so I don't believe
> this is a blocker. Fix it up if you like.
It's as you want, the rpmlint in rawhide does not care
> Other issues:
> I can't review spec file naming.
> I can't review the source used to build the SRPM as none was provided.
> BuildRequires: perl is not permitted.
This line is straight from the fedora-rpmdevtools-1.4-2.fc5 perl template. Is
the template wrong ?
> The license does not seem to be GPL. I see only:
> The Perl TTF module is licensed under the Perl Artistic License.
Ok, my mistake, copied the licensing in dries rpm without checking
> I don't understand this comment in the %files section:
> # For arch-specific packages: vendorarch
This is a bit of fedora-rpmdevtools-1.4-2.fc5 perl template I should have snipped
> I can't install the resulting RPM:
>
> error: Failed dependencies:
> perl(Win32) is needed by perl-Font-TTF-0.37-1.noarch
> perl(Win32::Registry) is needed by perl-Font-TTF-0.37-1.noarch
>
> I believe this is the result of lib/Font/TTF/Win32.pm. I'm not sure what would
> be best to do here. You can fix it with a quick
>
> rm %{buildroot}/%{perl_vendorlib}/Font/TTF/Win32.pm
>
> in the %install section but I'm not completely sure if that's acceptable.
I agree on this, will do
> Another solution would be to postprocess the output of the dependency generator,
> but that's rather unpalatable as well (and more complicated than just deleting
> the file).
--
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.
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-Font-TTF (part of the dejavu-fonts toolchain)
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=180034
------- Additional Comments From roozbeh(a)farsiweb.info 2006-02-05 03:42 EST -------
Nicolas, would you please provide an SRPM?
--
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.
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-Font-TTF (part of the dejavu-fonts toolchain)
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=180034
tibbs(a)math.uh.edu changed:
What |Removed |Added
----------------------------------------------------------------------------
OtherBugsDependingO|163776 |163778
nThis| |
------- Additional Comments From tibbs(a)math.uh.edu 2006-02-04 20:27 EST -------
Generally you should supply a src.rpm; it makes the review process a bit easier
and some of the review items require having it.
My rpmlint complains:
W: perl-Font-TTF wrong-file-end-of-line-encoding
/usr/share/doc/perl-Font-TTF-0.37/README.TXT
and indeed README.TXT has CRLF endings. However, there are several packages
which include documentation files with DOS-style line endings so I don't believe
this is a blocker. Fix it up if you like.
Other issues:
I can't review spec file naming.
I can't review the source used to build the SRPM as none was provided.
BuildRequires: perl is not permitted.
The license does not seem to be GPL. I see only:
The Perl TTF module is licensed under the Perl Artistic License.
I don't understand this comment in the %files section:
# For arch-specific packages: vendorarch
I can't install the resulting RPM:
error: Failed dependencies:
perl(Win32) is needed by perl-Font-TTF-0.37-1.noarch
perl(Win32::Registry) is needed by perl-Font-TTF-0.37-1.noarch
I believe this is the result of lib/Font/TTF/Win32.pm. I'm not sure what would
be best to do here. You can fix it with a quick
rm %{buildroot}/%{perl_vendorlib}/Font/TTF/Win32.pm
in the %install section but I'm not completely sure if that's acceptable.
Another solution would be to postprocess the output of the dependency generator,
but that's rather unpalatable as well (and more complicated than just deleting
the file).
--
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.
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-Font-TTF (part of the dejavu-fonts toolchain)
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=180034
tibbs(a)math.uh.edu changed:
What |Removed |Added
----------------------------------------------------------------------------
AssignedTo|bugzilla-sink(a)leemhuis.info |tibbs(a)math.uh.edu
--
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.
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-Font-TTF (part of the dejavu-fonts toolchain)
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=180034
Nicolas.Mailhot(a)laPoste.net changed:
What |Removed |Added
----------------------------------------------------------------------------
Platform|All |noarch
--
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.
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-Font-TTF (part of the dejavu-fonts toolchain)
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=180034
Nicolas.Mailhot(a)laPoste.net changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |fedora-perl-devel-
| |list(a)redhat.com
--
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.
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: spamassassin restart fails - functions bug?
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=161785
------- Additional Comments From mitr(a)redhat.com 2006-02-03 01:42 EST -------
FWIW, killproc and other initscript functions now support -p to specify the
PID file location, so the symlink hack should no longer be necessary.
--
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.
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: amavisd-new
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=167354
steve(a)silug.org changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |CLOSED
Resolution| |NEXTRELEASE
------- Additional Comments From steve(a)silug.org 2006-02-02 22:30 EST -------
Please open a new bug if you can reproduce the problem with 2.3.3-5 when it
comes out of the build system.
--
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.