Bug#742409: [Sereal] 2.06 fails to build on some architectures: (#47)

Niko Tyni ntyni at debian.org
Fri Sep 26 10:53:23 UTC 2014


On Fri, Sep 26, 2014 at 12:40:12PM +0200, demerphq wrote:

> You are definitely my favourite person of the week. Thank you SO MUCH. I
> stared at that source for hours looking for this, and I missed it every
> time.

Yeah, I needed quite a few hours with gdb to find it. Code glaring wasn't
working for me either :)

> > On Wed, Sep 17, 2014 at 03:57:24PM +0200, demerphq wrote:
> > > On 17 September 2014 07:52, Niko Tyni <ntyni at debian.org> wrote:

> > Finally found it. Can't see how it could have worked on big endian
> > 64 bit HPUX either as csnappy_internal_userspace.h seems to define
> > __LITTLE_ENDIAN there too. But never mind.
> >
> 
> Jarkko wasn't certain if we had made a 64 bit or 32 bit hpux big-endian
> work, so I bet I was just wrong there.
> 
> But, does this mean there is something else we need to fix?

No, I expect it's fine now. The __hpux branch in
csnappy_internal_userspace.h looks perfectly compatible with this to me.

> Let me roll a new release and we can celebrate together a debian clean
> build list.

OK; I expect I or somebody else from the Debian Perl group will pull it
in Debian quite quickly :)

> This is really really awesome. Thank you again. I can't tell you how
> pleased I am that this works on everything.

Thank you for all your work on this stuff!
-- 
Niko



More information about the pkg-perl-maintainers mailing list