Bug 249135

Summary: [PATCH] Improve perl.prov version extraction
Product: [Fedora] Fedora Reporter: Ville Skyttä <ville.skytta>
Component: rpmAssignee: Panu Matilainen <pmatilai>
Status: CLOSED UPSTREAM QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: low Docs Contact:
Priority: low    
Version: rawhideKeywords: Patch
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2007-07-23 09:30:39 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
Attachments:
Description Flags
Support fully qualified $Foo::Bar::VERSION expressions none

Description Ville Skyttä 2007-07-21 11:06:16 UTC
perl.prov does not support extraction of versions from fully qualified
expressions like "$Foo::Bar::VERSION = 1.00".  The attached patch (against
current rpm.org hg tip) fixes it.

This is a part of a bigger patch originally filed in bug 61797 ages ago.  The
patch there contains some things that need a review, but this part is the most
important part of it, should be safe and would help with majority of cases
listed in http://www.redhat.com/archives/fedora-maintainers/2007-July/msg00312.html

Comment 1 Ville Skyttä 2007-07-21 11:06:16 UTC
Created attachment 159716 [details]
Support fully qualified $Foo::Bar::VERSION expressions

Comment 2 Ville Skyttä 2007-07-21 11:11:23 UTC
One test case: current perl-Config-General in Fedora devel, before applying this
patch, it Provides:

    perl(Config::General)
    perl(Config::General::Extended)
    perl(Config::General::Interpolated)
    perl-Config-General = 2.33-1.fc8

After applying the patch and rebuilding:

    perl(Config::General) = 2.33
    perl(Config::General::Extended) = 2.02
    perl(Config::General::Interpolated) = 2.08
    perl-Config-General = 2.33-1.fc8


Comment 3 Jeff Johnson 2007-07-21 11:59:34 UTC
Added to rpm5.org cvs.

FWIW, perldeps, not perl.prov, will be used eventually so that both provides and requires extraction is in a 
single file to decrease /usr/lib/rpm litter, and should be patched as well.

Comment 4 Jeff Johnson 2007-07-21 12:11:56 UTC
Also FWIW, #61797 was filed around the time Chip Turner was writing perldeps. Again, perldeps, not 
perl.prov, needs whatever patches you attempt.

Comment 5 Ville Skyttä 2007-07-21 12:41:33 UTC
While perldeps.pl looks much cleaner than perl.req/perl.prov, it seems to me
that it does not have any module version handling whatsoever (only the version
of perl is supported), so this patch isn't really applicable to it at the moment.

And FWIW, in my opinion as long as module versions are not supported, switching
from perl.req/perl.prov to perldeps.pl should not be considered - it'd be a
largish regression.

Comment 6 Panu Matilainen 2007-07-23 09:30:39 UTC
In rpm.org now as well.

Just holler if you want to see this in Fedora rpm before it gets there through
new upstream release.

Comment 7 Ville Skyttä 2007-07-23 14:18:25 UTC
Thanks.  The sooner this is in, the sooner can the perl packages affected by
this issue be "fixed" by just rebuilding them.  But it's not that crucial -
we've managed just fine without this since the beginning of time.  I don't think
it would hurt to get it done for F8 though.