distcc build fail in hosts.c

[GoogleCodeExporter]

After fixing issue 3, building r456 still fails for me on Ubuntu Dapper.

Looks like you need to add @ZEROCONF_DISTCC_OBJS@ to $common_obj in
Makefile.in??

Console output:

gcc -g -O2 -MD -W -Wall -Wimplicit -Wuninitialized -Wshadow -Wpointer-arith
-Wcast-align -Wwrite-strings -Waggregate-return -Wstrict-prototypes
-Wmissing-prototypes -Wnested-externs -Wmissing-declarations -D_REENTRANT 
   -o distccd src/access.o src/daemon.o  src/dopt.o src/dparent.o
src/dsignal.o src/ncpus.o src/prefork.o src/stringmap.o src/serve.o
src/setuid.o src/srvnet.o src/srvrpc.o src/state.o src/stats.o
src/fix_debug_info.o src/zeroconf-reg.o src/arg.o src/argutil.o
src/cleanup.o src/compress.o src/trace.o src/util.o src/io.o src/exec.o
src/rpc.o src/tempfile.o src/bulk.o src/help.o src/filename.o src/lock.o
src/netutil.o src/pump.o src/sendfile.o src/safeguard.o src/snprintf.o
src/timeval.o src/dotd.o src/hosts.o src/hostfile.o src/implicit.o
src/loadfile.o src/gcc-id.o lzo/minilzo.o  -lpthread   -lpopt
-lavahi-common -lavahi-client
src/hosts.o: In function `dcc_parse_hosts':
/home/thestig/distcc/pump/distcc-read-only/src/hosts.c:516: undefined
reference to `dcc_zeroconf_add_hosts'
collect2: ld returned 1 exit status
make: *** [distccd] Error 1

Original issue reported on code.google.com by zhang...@gmail.com on 3 Jun 2008 at 9:37

[GoogleCodeExporter]

and remove @ZEROCONF_DISTCC_OBJS@ from $distcc_obj...

Original comment by zhang...@gmail.com on 3 Jun 2008 at 9:53

[GoogleCodeExporter]

No, I think it's happy in distccd_objs.  We need different zeroconf files in 
distcc
and distccd, so it's right not to have a var in common_objs, which is shared by 
both.

I think we'll need to install avahi to make sure we have everything fixed 
right..

Original comment by csilv...@gmail.com on 3 Jun 2008 at 9:56

• Changed state: Accepted

[GoogleCodeExporter]

I ended up adding some files to common_objs and some to distccd_objs.  Fixed in 
svn 
revision 457.

Original comment by csilv...@gmail.com on 3 Jun 2008 at 11:07

• Changed state: Fixed