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

[demerphq]

On 26 September 2014 12:53, Niko Tyni <ntyni at debian.org> wrote:

> 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 :)
>
>
Well thanks a ton for your efforts.

We will push your patch upstream to the Snappy project we forked from so
others can benefit from your effort as well.


> > > 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.
>
>
Great.


> > 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 :)
>
>
I just pushed 3.002_001 out to CPAN.


> > 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!
>

Ditto. Its been nice working with you Debian folks, a real pleasure to get
this taken care of.

Yves

-- 
perl -Mre=debug -e "/just|another|perl|hacker/"
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.alioth.debian.org/pipermail/pkg-perl-maintainers/attachments/20140926/ccbbda8d/attachment.html>