Build failure on OSX

Bug #443244 reported by Lee Bieber
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
libmemcached
Fix Released
Undecided
Trond Norbye

Bug Description

 /usr/bin/gcc-4.2 -std=gnu99 -DHAVE_CONFIG_H -I.. -I.. -ggdb3 -I/usr/local/include -pipe -O3 -Werror -pedantic -Wall -Wextra -Wundef -Wshadow -fdiagnostics-show-option -fvisibility=hidden -Wformat=2 -Wstrict-aliasing -Wstrict-prototypes -Wmissing-prototypes -Wredundant-decls -Wmissing-declarations -Wcast-align -Wswitch-default -Wswitch-enum -Wwrite-strings -D_THREAD_SAFE -MT protocol/binary_handler.lo -MD -MP -MF protocol/.deps/binary_handler.Tpo -c protocol/binary_handler.c -fno-common -DPIC -o protocol/.libs/binary_handler.o
cc1: warnings being treated as errors
protocol/binary_handler.c: In function 'get_response_handler':
protocol/binary_handler.c:129: warning: initialized field overwritten
protocol/binary_handler.c:129: warning: (near initialization for 'response.message')
make[3]: *** [protocol/binary_handler.lo] Error 1
make[2]: *** [all] Error 2
make[1]: *** [all-recursive] Error 1
make: *** [all] Error 2

% bzr version-info
revision-id: brian@gaz-20091002001112-jqdz0rqpmjr1y6oi
date: 2009-10-01 17:11:12 -0700
build-date: 2009-10-05 09:23:14 -0700
revno: 590
branch-nick: libmemcached

Related branches

Changed in libmemcached:
status: New → Confirmed
assignee: nobody → Trond Norbye (trond-norbye)
Revision history for this message
Lee Bieber (kalebral-deactivatedaccount) wrote :

Also broken on Ubuntu

Revision history for this message
Jay Pipes (jaypipes) wrote :
Revision history for this message
Trond Norbye (trond-norbye) wrote :

This is a bug in struct initializations in gcc (I actually had to rewrite the intialization pieces once due to a bug in gcc 4.4 with -Wmissing-field-initializers)

Guess we cannot assume that people have a working C99 compiler so we need to rewrite the code so that it works for broken compilers as well...

Changed in libmemcached:
status: Confirmed → Fix Committed
Revision history for this message
Trond Norbye (trond-norbye) wrote :

Released in revno: 591 [merge]

Changed in libmemcached:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.