Bug 671389

Summary: perl-RPM2 cannot be compiled against rpm-4.9.0 -> package RPM4
Product: [Fedora] Fedora Reporter: Petr Pisar <ppisar>
Component: perl-RPM2Assignee: Lubomir Rintel <lkundrak>
Status: CLOSED NEXTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: dmorganec, lkundrak, misc, mmaslano, perl-devel
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2011-02-08 19:36:04 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Bug Depends On: 675963    
Bug Blocks:    

Description Petr Pisar 2011-01-21 13:45:33 UTC
rpm-4.9.0_beta1 Changelog:

    - Since rpm 4.1 times, callers are supposed to use the rpmtsFooDB()
      variants instead of the lower-level rpmdb functions, and mixing
      them is not safe. Enforce the use of ts-level versions by making
      rpmdbLink(), rpmdbOpen(), rpmdbInit(), rpmdbClose(),
      rpmdbSync(), rpmdbVerify() and rpmdbRebuild() internal-only.

Thus perl-RPM2 cannot be linked at run-time:

Can't load 'blib/arch/auto/RPM2/RPM2.so' for module RPM2: blib/arch/auto/RPM2/RPM2.so: undefined symbol: rpmdbOpen at /usr/lib64/perl5/DynaLoader.pm line 200.

perl-RPM2 uses the old low level function only.

Should we drop this package in favor of (not yet packaged) RPM4?

Comment 1 Petr Pisar 2011-01-21 13:54:49 UTC
RPM4 does compile neither :(

Comment 2 D Morgan 2011-02-05 20:37:48 UTC
we need this rpm in mageia too, as perl-RPM4 maintainer is a mageia contributor, if we pass to make it compiles first, i will paste you the link here ( and if i forgot ( which can happen ;) ) the svn is here: http://viewvc.mageia.org/packages/cauldron/perl-RPM4/current/SOURCES/

Comment 3 Marcela Mašláňová 2011-02-07 09:19:13 UTC
Did you notice there is needed MDV::Packdrakeng? Isn't MDV shortcut for Mandriva? I'd rather don't push Mandriva's specific modules into our distro. I can look at RPM4 next week (if I don't forget ;-) ).

Comment 4 D Morgan 2011-02-08 11:33:34 UTC
Hi,

here is a patch done by Panu that allow to build with rpm 4.8:

 
http://viewvc.mageia.org/packages/cauldron/perl-RPM4/current/SOURCES/RPM4-0.23-rpm48.patch?revision=48941&view=markup

for MDV::Packdrakeng i don't think that this is mandriva's specific, this have been done for mdv thus the MDV Namespace. TO have a comparison this is like s-c-p that have dbus files with redhat in the name, that does prevent other distro to use it :).

Comment 5 Michael S. 2011-02-08 11:45:14 UTC
Well, while the naming doesn't affect the fact it can be distributed, and while I can understand the branding issue ( but would not consider blocker as there is also others perl packages with such namespace, like Danga::Socket ), the lack of clear tarball may be more annoying. 

That's something we plan to address in the Mageia project, and also we will try to upload to CPAN if it make sense. 

And that wouldn't be the first Mandriva software in Fedora, there is monitor-edid, prcsys, xsettings-kde.

Comment 6 Marcela Mašláňová 2011-02-08 12:26:00 UTC
(In reply to comment #5)
> Well, while the naming doesn't affect the fact it can be distributed, and while
> I can understand the branding issue ( but would not consider blocker as there
> is also others perl packages with such namespace, like Danga::Socket ), the
> lack of clear tarball may be more annoying. 
> 
> That's something we plan to address in the Mageia project, and also we will try
> to upload to CPAN if it make sense. 
> 
> And that wouldn't be the first Mandriva software in Fedora, there is
> monitor-edid, prcsys, xsettings-kde.

Nothing against Mandriva software. I was looking for some Mandriva specific behaviour or something like that, but it looks good. Anyway, it's new package and it will need review.

Comment 7 Marcela Mašláňová 2011-02-08 12:26:58 UTC
(In reply to comment #4)
> Hi,
> 
> here is a patch done by Panu that allow to build with rpm 4.8:
> 
> 
> http://viewvc.mageia.org/packages/cauldron/perl-RPM4/current/SOURCES/RPM4-0.23-rpm48.patch?revision=48941&view=markup
> 
Would be nice to have this patch also in upstream. Do you send it to them?

Comment 8 D Morgan 2011-02-08 12:30:40 UTC
(In reply to comment #7)
> (In reply to comment #4)
> > Hi,
> > 
> > here is a patch done by Panu that allow to build with rpm 4.8:
> > 
> > 
> > http://viewvc.mageia.org/packages/cauldron/perl-RPM4/current/SOURCES/RPM4-0.23-rpm48.patch?revision=48941&view=markup
> > 
> Would be nice to have this patch also in upstream. Do you send it to them?

not yet but i am in touch with the upstream dev and i planned to give him this patch.

Comment 9 D Morgan 2011-02-08 12:48:22 UTC
btw first i need to understand why regression tests are failing before asking for a new release.

Comment 10 Marcela Mašláňová 2011-02-08 13:01:33 UTC
Ok, first package prepared for review #675963.

Comment 11 Marcela Mašláňová 2011-02-08 13:15:50 UTC
Hm, if you are able build RPM4, you might help me with http://mmaslano.fedorapeople.org/RPM4/

I wasn't successful with Panu's patch, neither with my change. I'm using F-14 with rpm-4.8.1.

Comment 13 Lubomir Rintel 2011-02-08 19:36:04 UTC
IM IN UR BUGZILLA STEALIN UR TIKETZ

0.69 which was just uploaded to CPAN fixes this. I've updated the package as well, so I guess this can be closed now.