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