Bug 671389 - perl-RPM2 cannot be compiled against rpm-4.9.0 -> package RPM4
Summary: perl-RPM2 cannot be compiled against rpm-4.9.0 -> package RPM4
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: perl-RPM2
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Lubomir Rintel
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 675963
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-01-21 13:45 UTC by Petr Pisar
Modified: 2011-02-08 19:36 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-02-08 19:36:04 UTC
Type: ---


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
CPAN 64984 0 None None None Never

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.


Note You need to log in before you can comment on or make changes to this bug.