[Apt-Rpm] [PATCH] Make it build with rpm 5.2 again

Panu Matilainen pmatilai at laiskiainen.org
Sat Dec 13 02:17:17 PST 2008


On Tue, 18 Nov 2008, Bernhard Rosenkraenzer wrote:

> Hi,
> the recent changes in git broke rpm 5.2 support - the
> attached
> patch fixes it (I hope it doesn't break rpm 4.x -- in theory it
> shouldn't, but I don't have an rpm4 installation to check on).

Please change this to use "capability" checks instead. Rpm versions don't 
mean a thing these days, the only semi-safe range for rpm versions is < 
4.4.2 and even there differences exist as vendors have patched rpm all 
over.

One simple possibility would seem to be checking for rpm4compat.h 
existence, rpm.org uses rpmlegacy.h for similar purpose so it's easy to 
distinguish between the two.

Another thing is that I'd like to see these compatibility #ifdef's grouped 
into one place as much as possible to get them out of sight. rapttypes.h 
is a start towards that direction but obviously a LOT more work is needed, 
the compatibility cruft has spread all over apt-pkg/rpm/ directory over 
the years.

Then of course apt-rpm can't be stuck forever on the legacy 
compatibility wrappers but that's another story...

 	- Panu -



More information about the Apt-Rpm mailing list