On 26 September 2014 12:53, Niko Tyni <nt...@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 <nt...@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/"