[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#618488: dpkg-cross libidl fix



+++ Neil Williams [2011-03-15 22:29 +0000]:
> On Tue, 15 Mar 2011 15:29:44 +0000
> Wookey <wookey@wookware.org> wrote:
> 
> 
> > I just checked and that is now right.
> 
> But will then break for anyone building older versions of the package.

Indeed, but dpkg-cross is normally going to be used to build the
version in the distro. I don't know when this changed, but the new
answr is likely more useful than the old one for sensible situations. 

> > i.e the % and the u are now not part of libIDL_cv_long_long_format
> > 
> > can you fix that upstream? I'll fix it in the ubuntu version.
> 
> The change may need to have a version check sorted out somewhere around:
> if [ "$PACKAGE" = "libidl" -o "$PACKAGE_NAME" = "libIDL" ]; then

I wondered if we could supply values for different versions. That
would indeed be more correct (except that I have no idea what cutoff
version to use). 

> In other words, it's about time that this change and all the others
> like it, get pushed upstream and handled properly.

How would this be handled upstream? For a configure-time compiled test
which needs to be run on the host arch dpkg-cross cached supply of the
answers is correct is it not?

> The value we do have is probably only usable for ARM anyway.

I did wonder about that. It's the same on amd64 so 'll' may be quite
widespread. The other options or 'i' and 'I64' which may well only
apply on strange systems with funny C-libraries.


Wookey
-- 
Principal hats:  Linaro, Emdebian, Wookware, Balloonboard, ARM
http://wookware.org/



Reply to: