On Mon, Oct 05, 2015 at 01:40:16PM +0200, Jrmie Courr??ges-Anglas wrote: > Stuart Henderson <st...@openbsd.org> writes: > > > On 2015/10/04 22:41, Mikolaj Kucharski wrote: > >> On Sun, Oct 04, 2015 at 10:15:12PM +0100, Mikolaj Kucharski wrote: > >> > I would appreciate if someone look is my change sane. As this is long > >> > overdue update from 0.4 to 0.6 which has also different upstream, I > >> > decided to include whitespace changes in Makefile, as this update is > >> > intrusive. > >> > >> - switch to sed -i > >> - fix regress tests > >> - all regress tests pass > > > > Looks good to me. The patch-README diff in your mail doesn't apply > > directly, but it's just 'cvs rm' anyway so that's easy to fix. > > > > Would you be interested in taking MAINTAINER? > > FWIW, looks good to me too, ok jca@ > > Mikolaj: note that upstream already has AC_C_BIGENDIAN in its > configure.ac and already uses it... *except* when the system has an > <endian.h> include file. That special case is where the bug was > introduced; what was the reason for this special case?
I would need to look into this, as at the moment I'm not sure what do you mean. I'm not that familiar with autotools magic. I'm not upstream, so I don't know what was the reason for this special case. > Which reminds me that some versions of AC_C_BIGENDIAN are broken on > powerpc... -- best regards q#