squid 3.0.STABLE6-2 package failing to build.

Bug #248318 reported by Amos Jeffries on 2008-07-14
4
Affects Status Importance Assigned to Milestone
squid3 (Ubuntu)
Undecided
Michael Bienia

Bug Description

Binary package hint: squid3

We (upstream) have a patch for this at http://www.squid-cache.org/Versions/v3/3.0/changesets/b8856.patch

Related branches

Amos Jeffries (yadi) wrote :

Forgot to mention. Will be fixed in STABLE8. Any further issue please let me know.

Michael Bienia (geser) wrote :

I've applied that patch on squid3 3.0.STABLE7-1 (taken from Debian unstable) and tried to build it in Ubuntu intrepid. Unfortunately it still fails to build with the same error (I checked that the patch got really applied):

cc -DHAVE_CONFIG_H -I. -I. -I../include -I../include -I../include -I/usr/include -Werror -Wall -Wpointer-arith -Wwrite-strings -Wmissing-prototypes -Wmissing-declarations -Wcomments -D_REENTRANT -m64 -g -O2 -g -Wall -O2 -c util.c
cc1: warnings being treated as errors
util.c: In function 'default_failure_notify':
util.c:934: error: ignoring return value of 'write', declared with attribute warn_unused_result
util.c:935: error: ignoring return value of 'write', declared with attribute warn_unused_result
make[3]: *** [util.o] Error 1
make[3]: Leaving directory `/tmp/buildd/squid3-3.0.STABLE7/lib'

Changed in squid3:
assignee: nobody → geser
status: New → Confirmed

Michael Bienia wrote:
> I've applied that patch on squid3 3.0.STABLE7-1 (taken from Debian
> unstable) and tried to build it in Ubuntu intrepid. Unfortunately it
> still fails to build with the same error (I checked that the patch got
> really applied):

Darn. Thank you. Looks like we may have to go for a less elegant hack.

Does it work (or at least skip to a later problem) with that patch and
this attached one on top?

Amos

>
> cc -DHAVE_CONFIG_H -I. -I. -I../include -I../include -I../include -I/usr/include -Werror -Wall -Wpointer-arith -Wwrite-strings -Wmissing-prototypes -Wmissing-declarations -Wcomments -D_REENTRANT -m64 -g -O2 -g -Wall -O2 -c util.c
> cc1: warnings being treated as errors
> util.c: In function 'default_failure_notify':
> util.c:934: error: ignoring return value of 'write', declared with attribute warn_unused_result
> util.c:935: error: ignoring return value of 'write', declared with attribute warn_unused_result
> make[3]: *** [util.o] Error 1
> make[3]: Leaving directory `/tmp/buildd/squid3-3.0.STABLE7/lib'
>
>
> ** Changed in: squid3 (Ubuntu)
> Assignee: (unassigned) => Michael Bienia (geser)
>
> ** Changed in: squid3 (Ubuntu)
> Status: New => Confirmed
>

--
Please use Squid 2.7.STABLE3 or 3.0.STABLE7

Michael Bienia (geser) wrote :

Yes, util-suse-fix.patch makes the build go a little bit further.

It now stops at:

Making all in src
make[1]: Entering directory `/tmp/buildd/squid3-3.0.STABLE7/src'
/bin/bash ../libtool --tag=CXX --mode=compile g++ -DHAVE_CONFIG_H -DDEFAULT_CONFIG_FILE=\"/etc/squid3/squid.conf\" -I. -I. -I../include -I. -I. -I../include -I../include -I../lib/libTrie/include -I/usr/include -Werror -Wall -Wpointer-arith -Wwrite-strings -Wcomments -D_REENTRANT -m64 -g -O2 -g -Wall -O2 -c -o comm.lo comm.cc
 g++ -DHAVE_CONFIG_H -DDEFAULT_CONFIG_FILE=\"/etc/squid3/squid.conf\" -I. -I. -I../include -I. -I. -I../include -I../include -I../lib/libTrie/include -I/usr/include -Werror -Wall -Wpointer-arith -Wwrite-strings -Wcomments -D_REENTRANT -m64 -g -O2 -g -Wall -O2 -c comm.cc -o comm.o
cc1plus: warnings being treated as errors
In file included from comm.cc:38:
StoreIOBuffer.h: In member function 'void StoreIOBuffer::dump() const':
StoreIOBuffer.h:71: error: ignoring return value of 'size_t fwrite(const void*, size_t, size_t, FILE*)', declared with attribute warn_unused_result
StoreIOBuffer.h:72: error: ignoring return value of 'size_t fwrite(const void*, size_t, size_t, FILE*)', declared with attribute warn_unused_result
comm.cc: In static member function 'static void CommIO::Initialise()':
comm.cc:2223: error: ignoring return value of 'int pipe(int*)', declared with attribute warn_unused_result
make[1]: *** [comm.lo] Error 1
make[1]: Leaving directory `/tmp/buildd/squid3-3.0.STABLE7/src'
make: *** [all-recursive] Error 1

Amos Jeffries (yadi) wrote :

Michael Bienia wrote:
> Yes, util-suse-fix.patch makes the build go a little bit further.
>

Excellent.

Here is a patch that should fully supersede both b8856 and the small
experimental one.
If there are no other problems. I'll make this the formal fix.

Amos
--
Please use Squid 2.7.STABLE3 or 3.0.STABLE7

Michael Bienia (geser) wrote :

Thanks, the last patch (b8856-updated) worked. squid3 build successfully.

Launchpad Janitor (janitor) wrote :

This bug was fixed in the package squid3 - 3.0.STABLE7-1ubuntu1

---------------
squid3 (3.0.STABLE7-1ubuntu1) intrepid; urgency=low

  * Apply upstream patch to fix errors about unused return values
    (fixes a FTBFS; LP: #248318).
    Thanks to Amos Jeffries for providing the patch.
  * debian/control:
    + Modify Maintainer value to match DebianMaintainerField spec.

squid3 (3.0.STABLE7-1) unstable; urgency=low

  * New upstream release

 -- Michael Bienia <email address hidden> Tue, 15 Jul 2008 11:26:02 +0200

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

Other bug subscribers