Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Misnamed package, add Provides?
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=165885
------- Additional Comments From ville.skytta(a)iki.fi 2005-08-14 05:05 EST -------
I thought that the package naming guidelines would have adviced to use
perl-$CPANDIST, but instead it goes on to say things about "the CPAN module
name".
IMO, that makes no sense because the unit of CPAN module packaging is a CPAN
_distribution_, and the vast majority those distributions contain more than
one module. What's "the CPAN module name" then? Just using the CPAN
distribution name would work, and is already used in all CPAN packages I'm
aware of.
--
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: Misnamed package, add Provides?
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=165885
rc040203(a)freenet.de changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |fedora-perl-devel-
| |list(a)redhat.com
------- Additional Comments From rc040203(a)freenet.de 2005-08-14 00:00 EST -------
(In reply to comment #0)
> The Perl module package naming convention is perl-$CPANDIST, and this package
> doesn't follow it (should be perl-gettext, not perl-Locale-gettext).
Is it? Is this documented anywhere?
> Now, this is very much my fault, didn't catch it in the review, and it doesn't
> really matter much as all dependencies should be specified as
> perl(Locale::gettext) anyway.
Exactly.
But I think the actual problem is in CPAN:
The tarball is called "gettext-*.tar.gz", while the actual module is
"Locale::gettext".
I chose perl-Locale-gettext for the RPM, primarily because I assumed people will
be looking for perl-Locale-gettext-*.rpm when encountering a perl dependency on
Locale::gettext.
Also, I find calling the RPM perl-gettext*.rpm to be misleading because it would
indicate this module to contain Perl's "one and only gettext binding". However
this is just _one_ aribitrary gettext binding.
> I thought it would be nice to add "Provides:
> perl-gettext = %{version}-%{release}" into this package nevertheless.
I am not opposed to it, if people think this is useful.
> No strong opinions though; add the provides, or rename the package, or leave
> it as is. Your call.
Neither strong opinions by me, too.
Advice appreciated
--
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: perl: specfile cleanup and update to 5.8.7
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=165907
wtogami(a)redhat.com changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |altblue(a)n0i.net
------- Additional Comments From wtogami(a)redhat.com 2005-08-13 22:05 EST -------
*** Bug 158383 has been marked as a duplicate of this bug. ***
--
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: Perl 5.8.7 is out
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=158383
wtogami(a)redhat.com changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |CLOSED
Resolution| |DUPLICATE
OtherBugsDependingO|150222 |
nThis| |
------- Additional Comments From wtogami(a)redhat.com 2005-08-13 22:05 EST -------
*** This bug has been marked as a duplicate of 165907 ***
--
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: perl: specfile cleanup and update to 5.8.7
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=165907
------- Additional Comments From wtogami(a)redhat.com 2005-08-13 19:55 EST -------
Isn't it a good idea to wait until the third patch is complete before pushing
this to rawhide? I want to avoid the case of the version of obsoleted
perl-subpackage modules going backwards temporarily.
--
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: perl: specfile cleanup and update to 5.8.7
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=165907
------- Additional Comments From jpo(a)di.uminho.pt 2005-08-13 18:00 EST -------
The file in comment #4 is just a precaution because:
* all perl versions 5.8 have been shipping with Filter::Simple version 0.78
Output from "corelist -a Filter::Simple"
perl Filter::Simple
...
5.008 0.78
5.008001 0.78
...
5.008006 0.78
5.008007 0.78
...
* Fedora shipped Filter::Simple 0.79 as an external package
(FC4: perl-Filter-Simple-0.79)
* there is no change to the module between version 0.78 and 0.79 (expect the
the version)
Diff from Filter-Simple-0.78 to Filter-Simple-0.79
http://search.cpan.org/diff?from=Filter-Simple-0.78&to=Filter-Simple-0.79
--
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: perl: specfile cleanup and update to 5.8.7
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=165907
------- Additional Comments From jpo(a)di.uminho.pt 2005-08-13 17:51 EST -------
The second patch no longer deletes the following core modules:
* Filter::Util::Call (external package perl-Filter)
* Filter::Simple (external package perl-Filter-Simple)
* Time::HiRes (external package perl-Time-HiRes)
and obsoletes the following FC-4 core packages:
* perl-Filter
* perl-Filter-Simple
* perl-Time-HiRes
Notes:
* the find command with the globbing expression *Filter* also removed
the DBM_Filter module. I changed the above globbing to *DBM_Filter*
in order to keep the previous behaviour.
--
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: perl: specfile cleanup and update to 5.8.7
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=165907
------- Additional Comments From jpo(a)di.uminho.pt 2005-08-13 17:35 EST -------
Current status:
The first patch has already been applied (see perl-5.8.6-16.src.rpm in rawhide)
First patch summary: perl-5.8.6-15 => perl-5.8.6-16
===================================================
* Specfile reformatted
- in particular grouped all the patches together
* Full URL of the perl 5.8.6 tarball
* %build section
- moved the mkdir commands that appeared before the
Configure line into the %install section
- Configure command argument changes
-Dversion
-Dinstallprefix
(make install DESTDIR=$RPM_... in %install)
* %install section
- make install DESTDIR=$RPM_BUILD_ROOT
- no longer creates MANIFEST.all
- dropped compression of the man pages
* %files section
- no longer uses MANIFEST.all for the perl package
- no longer uses MANIFEST.suidperl for the perl-suidperl subpackage
* Source files no longer needed
- Source1: clean-manifest.pl
- Source9: MANIFEST.suidperl
- Source10: system-owned-directories
Other notes:
* didn't move "make test" into the %check section
Reason: the perl building process prints a message if the "make install"
step is started without having executed "make test" before
Wish/pending list:
* mark libnet.cfg as a configuration file?
maybe move it to /etc and create a symbolic link to it?
* reformat/clean the %install section
I don't have access to a x86_64 (or 64 bit) machine to test the
multilib stuff
* mark pod files as docs
should be done at the RPM level - other perl packages (and maybe
Ruby) would also profit from it
--
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.