Bug#742421: libhdf4: that fix won't quite work on Debian

Rebecca Palmer rebecca_palmer at zoho.com
Thu Sep 11 17:28:32 UTC 2014


(from debian/patches/hdfi.h at 
https://launchpad.net/ubuntu/+source/libhdf4/4.2r10-0ubuntu1 )
> --- a/HDF4/hdf/src/hconv.h
> +++ b/HDF4/hdf/src/hconv.h
> @@ -59,7 +59,7 @@
>  /* CONSTANT DEFINITIONS                                                      */
>  /*****************************************************************************/
>  /* Generally Big-Endian machines */
> -#if !defined(INTEL86) && !defined(MIPSEL) && !defined(DEC_ALPHA) && !defined(I860) && !defined(SUN386) && !(defined(__ia64) && !(defined(hpux) || defined(__hpux))) && !defined(__x86_64__)
> +#if !defined(INTEL86) && !defined(MIPSEL) && !defined(DEC_ALPHA) && !defined(I860) && !defined(SUN386) && !(defined(__ia64) && !(defined(hpux) || defined(__hpux))) && !defined(__x86_64__) && (defined(__powerpc__) && !defined(__LITTLE_ENDIAN__)) && !defined(__aarch64__) && !defined(__ARM_EABI__)

That treats anything that isn't powerpc as little-endian, which happens 
to be true in Ubuntu, but isn't in Debian.

If you want to keep this (weird, and fragile in that it silently treats 
anything it doesn't recognise as big-endian) form use

+#if !defined(INTEL86) && !defined(MIPSEL) && !defined(DEC_ALPHA) && 
!defined(I860) && !defined(SUN386) && !(defined(__ia64) && 
!(defined(hpux) || defined(__hpux))) && !defined(__x86_64__) && 
!(defined(__powerpc__) && defined(__LITTLE_ENDIAN__)) && 
!defined(__aarch64__) && !defined(__ARM_EABI__)

but if it were my package I'd probably do something similar to #761203 ( 
https://bugs.debian.org/cgi-bin/bugreport.cgi?msg=5;filename=naref_endianness.patch;att=1;bug=761203 
)



More information about the Pkg-grass-devel mailing list