Re: Bug#664470: proxytunnel: FTBFS on kfreebsd-* and hurd

2014-01-28 Thread Steven Chamberlain
unarchive 664470
reopen 664470 =
found 664470 1.9.0+svn250-1
tags 664470 + patch
thanks

Hi,

This bug was been reintroduced in 1.9.0+svn250-1.  My patch from
http://bugs.debian.org/664470#5 was dropped without explanation.  Please
find a refreshed copy of it attached, which fixes the FTBFS on
kfreebsd-* and hurd.

Thanks,
Regards,
-- 
Steven Chamberlain
ste...@pyro.eu.org
--- proxytunnel-1.9.0+svn250.orig/Makefile
+++ proxytunnel-1.9.0+svn250/Makefile
@@ -25,9 +25,6 @@
 # Most systems
 OPTFLAGS += -DSETPROCTITLE -DSPT_TYPE=2
 
-# Comment if you don't have this flag
-OPTFLAGS += -DSO_REUSEPORT
-
 # System dependant blocks... if your system is listed below, uncomment
 # the relevant lines
 


Re: Bug#664470: proxytunnel: FTBFS on kfreebsd-* and hurd

2014-01-28 Thread Julian Gilbey
On Tue, Jan 28, 2014 at 06:52:14PM +, Steven Chamberlain wrote:
 unarchive 664470
 reopen 664470 =
 found 664470 1.9.0+svn250-1
 tags 664470 + patch
 thanks
 
 Hi,
 
 This bug was been reintroduced in 1.9.0+svn250-1.  My patch from
 http://bugs.debian.org/664470#5 was dropped without explanation.  Please
 find a refreshed copy of it attached, which fixes the FTBFS on
 kfreebsd-* and hurd.

Oops, many apologies for this.  I hadn't remembered that it had been
commented out for this reason.

More significantly, it would be much better to do a test for this than
to universally unset this.  Uh, duh, SO_REUSEPORT is #define'd in
/usr/include/asm-generic/socket.h on Linux systems, which is included
(via sys/socket.h), so the simple test #ifdef SO_REUSEPORT in
proxytunnel.c works without having to manually define it in the
Makefile.

I've just uploaded a fixed version (-3).

   Julian


-- 
To UNSUBSCRIBE, email to debian-bsd-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/20140128202027.ga24...@d-and-j.net



Bug#664470: proxytunnel: FTBFS on kfreebsd-* and hurd

2012-03-17 Thread Steven Chamberlain
Package: src:proxytunnel
Version: 1.9.0-3
Tags: patch
User: debian-bsd@lists.debian.org
Usertags: kfreebsd
X-Debbugs-Cc: debian-bsd@lists.debian.org

Hi,

Something weird in the Makefile causes this error on kfreebsd and hurd
builds:

 cc -Wall -O2 -ggdb -DREV=0 -DHAVE_GETOPT_LONG -DUSE_SSL -DSETPROCTITLE 
 -DSPT_TYPE=2 -DSO_REUSEPORT -c -o proxytunnel.o proxytunnel.c
 In file included from /usr/include/i386-kfreebsd-gnu/sys/socket.h:40:0,
  from /usr/include/netinet/in.h:25,
  from proxytunnel.c:28:
 /usr/include/i386-kfreebsd-gnu/bits/socket.h:345:5: error: expected 
 identifier before numeric constant
 proxytunnel.c: In function ‘do_daemon’:
 proxytunnel.c:190:36: error: ‘SO_REUSEPORT’ undeclared (first use in this 
 function)
 proxytunnel.c:190:36: note: each undeclared identifier is reported only once 
 for each function it appears in
 make: *** [proxytunnel.o] Error 1

By defining SO_REUSEPORT in the compile options, the sys/socket.h header
is unable to properly define it.

I think that hack was originally meant to help with Cygwin builds.  But
I've tried removing it (please see attached patch), and that fixes the
build on kfreebsd-i386 and still builds fine on (Linux) amd64 and so
probably all of the Debian target architecures.  Please consider
applying it.

Thanks,
Regards,
-- 
Steven Chamberlain
ste...@pyro.eu.org
Index: proxytunnel-1.9.0/Makefile
===
--- proxytunnel-1.9.0.orig/Makefile	2012-03-18 00:28:46.0 +
+++ proxytunnel-1.9.0/Makefile	2012-03-18 00:28:56.0 +
@@ -16,9 +16,6 @@
 # Most systems
 OPTFLAGS += -DSETPROCTITLE -DSPT_TYPE=2
 
-# Comment if you don't have this flag
-OPTFLAGS += -DSO_REUSEPORT
-
 # System dependant blocks... if your system is listed below, uncomment
 # the relevant lines