Thanks, guys. Do you plan on releasing a new tarball with this fix? If not, what's the correct way to compile from the GitHub master?
On Wed, Oct 7, 2009 at 12:36 AM, Keith Rarick <[email protected]> wrote: > > I just pushed this fix. It works for me on Linux 32-bit and Mac OS > 10.5 32-bit. I have not tested Solaris yet. I think this means I > should add an autoconf check for C99. > > kr > > On Tue, Oct 6, 2009 at 11:00 PM, Yun Huang Yong <[email protected]> wrote: > > > > Jaume Sabater wrote: > >> I have downloaded the recently released version 1.4 of beanstalkd and > >> tried to compile it against libevent-1.4.2. When doing make, there is > >> an error that appears (it's actually a warning, but due to the -Werror > >> parameter it appears as error). > > > >> cc1: warnings being treated as errors > >> prot.c: In function ‘fmt_stats’: > >> prot.c:852: error: format ‘%llu’ expects type ‘long long unsigned > >> int’, but argument 9 has type ‘uint64_t’ > >> [..] > > > > I've tidied this up on http://github.com/goosmurf/beanstalkd and have > > submitted a pull request to Keith. > > > > Builds fine on my Linux amd64, hope it doesn't break other platforms. :D > > > > The issue is that there is a mix of usage of 'long long unsigned' and > > 'uint64_t' which whilst equivalent on most(?) platforms might be > > different on others. Fixed by ensuring all declarations are now > > uint64_t and using the PRIu64 format macro defined by inttypes.h. > > > > yun > > > > -- > > Yun Huang Yong > > [email protected] ...nom nom nom > > +61 408 131 419 > > -- > > > > > > > > > > > --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "beanstalk-talk" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/beanstalk-talk?hl=en -~----------~----~----~----~------~----~------~--~---
