Build failed in Hudson: 3.1-i386-Debian-sid #21

2009-09-26 Thread noc
See 

--
Started by user amos
Building remotely on rio.treenet
bzr: ERROR: Invalid http response for 
http://www.squid-cache.org/bzr/squid3/branches/SQUID_3_1/.bzr/branch-format: 
Bad status line received
Using saved parent location: 
http://www.squid-cache.org/bzr/squid3/branches/SQUID_3_1/
ERROR: Failed to pull



Build failed in Hudson: 3.1-i386-Debian-sid #20

2009-09-26 Thread noc
See 

--
Started by upstream project "3.1-amd64-CentOS-5.3" build number 24
Building remotely on rio.treenet
bzr: ERROR: Invalid http response for 
http://www.squid-cache.org/bzr/squid3/branches/SQUID_3_1/.bzr/branch-format: 
Bad status line received
Using saved parent location: 
http://www.squid-cache.org/bzr/squid3/branches/SQUID_3_1/
ERROR: Failed to pull



Build failed in Hudson: 3.1-i386-Debian-sid #19

2009-09-26 Thread noc
See 

--
Started by upstream project "3.1-amd64-CentOS-5.3" build number 23
Building remotely on rio.treenet
bzr: ERROR: Invalid http response for 
http://www.squid-cache.org/bzr/squid3/branches/SQUID_3_1/.bzr/branch-format: 
Bad status line received
Using saved parent location: 
http://www.squid-cache.org/bzr/squid3/branches/SQUID_3_1/
ERROR: Failed to pull



Build failed in Hudson: 3.HEAD-amd64-CentOS-5.3 #119

2009-09-26 Thread noc
See 

Changes:

[Amos Jeffries ] Prep for 3.1.0.14

[Automatic source maintenance ] SourceFormat 
Enforcement

--
[...truncated 483 lines...]
checking varargs.h usability... no
checking varargs.h presence... no
checking for varargs.h... no
checking byteswap.h usability... yes
checking byteswap.h presence... yes
checking for byteswap.h... yes
checking glib.h usability... no
checking glib.h presence... no
checking for glib.h... no
checking for stdint.h... (cached) yes
checking for inttypes.h... (cached) yes
checking for grp.h... (cached) yes
checking db.h usability... yes
checking db.h presence... yes
checking for db.h... yes
checking db_185.h usability... yes
checking db_185.h presence... yes
checking for db_185.h... yes
checking for linux/netfilter_ipv4.h... yes
checking for linux/netfilter_ipv4/ip_tproxy.h... no
checking for net/if.h... yes
checking for netinet/if_ether.h... yes
checking for netinet/icmp6.h... yes
checking for netinet/in.h... (cached) yes
checking for netinet/ip.h... yes
checking for netinet/ip6.h... yes
checking for netinet/ip_compat.h... no
checking for netinet/ip_fil_compat.h... (cached) no
checking for netinet/ip_fil.h... no
checking for netinet/ip_icmp.h... yes
checking for netinet/ipl.h... no
checking for netinet/ip_nat.h... no
checking for net/pf/pfvar.h... no
checking for net/pfvar.h... no
checking for sys/mount.h... yes
checking for resolv.h... yes
checking libxml/parser.h usability... no
checking libxml/parser.h presence... no
checking for libxml/parser.h... no
checking libxml/parser.h usability... yes
checking libxml/parser.h presence... yes
checking for libxml/parser.h... yes
checking for an ANSI C-conforming const... yes
checking whether byte ordering is bigendian... no
checking whether struct tm is in sys/time.h or time.h... time.h
checking for struct tm.tm_gmtoff... yes
checking for struct mallinfo... yes
checking for struct mallinfo.mxfast... no
checking for struct rusage... yes
checking for struct iphdr.ip_hl... yes
checking for void *... yes
checking size of void *... 8
checking for int8_t... yes
checking for int8_t... (cached) yes
checking size of int8_t... 1
checking for char... yes
checking for char... (cached) yes
checking size of char... 1
checking for u_int8_t... yes
checking for u_int8_t... (cached) yes
checking size of u_int8_t... 1
checking for uint8_t... yes
checking for uint8_t... (cached) yes
checking size of uint8_t... 1
checking for int16_t... yes
checking for int16_t... (cached) yes
checking size of int16_t... 2
checking for short... yes
checking for short... (cached) yes
checking size of short... 2
checking for int... yes
checking for int... (cached) yes
checking size of int... 4
checking for u_int16_t... yes
checking for u_int16_t... (cached) yes
checking size of u_int16_t... 2
checking for uint16_t... yes
checking for uint16_t... (cached) yes
checking size of uint16_t... 2
checking for int32_t... yes
checking for int32_t... (cached) yes
checking size of int32_t... 4
checking for long... yes
checking for long... (cached) yes
checking size of long... 8
checking for u_int32_t... yes
checking for u_int32_t... (cached) yes
checking size of u_int32_t... 4
checking for uint32_t... yes
checking for uint32_t... (cached) yes
checking size of uint32_t... 4
checking for int64_t... yes
checking for int64_t... (cached) yes
checking size of int64_t... 8
checking for __int64... no
checking for long long... yes
checking for long long... (cached) yes
checking size of long long... 8
checking for u_int64_t... yes
checking for u_int64_t... (cached) yes
checking size of u_int64_t... 8
checking for uint64_t... yes
checking for uint64_t... (cached) yes
checking size of uint64_t... 8
checking for pad128_t... no
checking for upad128_t... no
checking for pid_t... yes
checking for size_t... yes
checking for size_t... (cached) yes
checking size of size_t... 8
checking for ssize_t... yes
checking for off_t... yes
checking for off_t... (cached) yes
checking size of off_t... 8
checking for mode_t... yes
checking for fd_mask... yes
checking for working alloca.h... yes
checking for alloca... yes
checking for socklen_t... yes
yes
checking sys/capability.h usability... yes
checking sys/capability.h presence... yes
checking for sys/capability.h... yes
checking for operational libcap2... (cached) dummy
checking for mtyp_t... no
checking for main in -lnsl... yes
checking for main in -lsocket... no
checking for unix domain sockets... yes
checking for main in -lgnumalloc... no
checking for main in -lmalloc... no
checking for main in -lbsd... yes
checking for main in -lregex... no
checking for gethostbyname in -lbind... no
checking for main in -lresolv... yes
checking for main in -lm... yes
checking whether to enable IPv6... yes
checking if PF_INET6 is available... yes

checking for IPv6 split-stack requirement... no
checking for IPv6 v4-mapping ability... yes
checking for IPv6 stack ty

Build failed in Hudson: 3.1-i386-Debian-sid #18

2009-09-26 Thread noc
See 

--
Started by upstream project "3.1-amd64-CentOS-5.3" build number 22
Building remotely on rio.treenet
bzr: ERROR: Invalid http response for 
http://www.squid-cache.org/bzr/squid3/branches/SQUID_3_1/.bzr/branch-format: 
Bad status line received
Using saved parent location: 
http://www.squid-cache.org/bzr/squid3/branches/SQUID_3_1/
ERROR: Failed to pull



Build failed in Hudson: 3.HEAD-amd64-CentOS-5.3 #118

2009-09-26 Thread noc
See 

Changes:

[Automatic source maintenance ] SourceFormat 
Enforcement

[Henrik Nordstrom ] Make ESI parser modules expat 
and libxml2 dependent on their libraries

The ESI parser system is actually pluggable. There is no reason we should
require expat and libxml2. Just build what works.

Todo: Add an option to force the desired parsers.

--
[...truncated 482 lines...]
checking varargs.h usability... no
checking varargs.h presence... no
checking for varargs.h... no
checking byteswap.h usability... yes
checking byteswap.h presence... yes
checking for byteswap.h... yes
checking glib.h usability... no
checking glib.h presence... no
checking for glib.h... no
checking for stdint.h... (cached) yes
checking for inttypes.h... (cached) yes
checking for grp.h... (cached) yes
checking db.h usability... yes
checking db.h presence... yes
checking for db.h... yes
checking db_185.h usability... yes
checking db_185.h presence... yes
checking for db_185.h... yes
checking for linux/netfilter_ipv4.h... yes
checking for linux/netfilter_ipv4/ip_tproxy.h... no
checking for net/if.h... yes
checking for netinet/if_ether.h... yes
checking for netinet/icmp6.h... yes
checking for netinet/in.h... (cached) yes
checking for netinet/ip.h... yes
checking for netinet/ip6.h... yes
checking for netinet/ip_compat.h... no
checking for netinet/ip_fil_compat.h... (cached) no
checking for netinet/ip_fil.h... no
checking for netinet/ip_icmp.h... yes
checking for netinet/ipl.h... no
checking for netinet/ip_nat.h... no
checking for net/pf/pfvar.h... no
checking for net/pfvar.h... no
checking for sys/mount.h... yes
checking for resolv.h... yes
checking libxml/parser.h usability... no
checking libxml/parser.h presence... no
checking for libxml/parser.h... no
checking libxml/parser.h usability... yes
checking libxml/parser.h presence... yes
checking for libxml/parser.h... yes
checking for an ANSI C-conforming const... yes
checking whether byte ordering is bigendian... no
checking whether struct tm is in sys/time.h or time.h... time.h
checking for struct tm.tm_gmtoff... yes
checking for struct mallinfo... yes
checking for struct mallinfo.mxfast... no
checking for struct rusage... yes
checking for struct iphdr.ip_hl... yes
checking for void *... yes
checking size of void *... 8
checking for int8_t... yes
checking for int8_t... (cached) yes
checking size of int8_t... 1
checking for char... yes
checking for char... (cached) yes
checking size of char... 1
checking for u_int8_t... yes
checking for u_int8_t... (cached) yes
checking size of u_int8_t... 1
checking for uint8_t... yes
checking for uint8_t... (cached) yes
checking size of uint8_t... 1
checking for int16_t... yes
checking for int16_t... (cached) yes
checking size of int16_t... 2
checking for short... yes
checking for short... (cached) yes
checking size of short... 2
checking for int... yes
checking for int... (cached) yes
checking size of int... 4
checking for u_int16_t... yes
checking for u_int16_t... (cached) yes
checking size of u_int16_t... 2
checking for uint16_t... yes
checking for uint16_t... (cached) yes
checking size of uint16_t... 2
checking for int32_t... yes
checking for int32_t... (cached) yes
checking size of int32_t... 4
checking for long... yes
checking for long... (cached) yes
checking size of long... 8
checking for u_int32_t... yes
checking for u_int32_t... (cached) yes
checking size of u_int32_t... 4
checking for uint32_t... yes
checking for uint32_t... (cached) yes
checking size of uint32_t... 4
checking for int64_t... yes
checking for int64_t... (cached) yes
checking size of int64_t... 8
checking for __int64... no
checking for long long... yes
checking for long long... (cached) yes
checking size of long long... 8
checking for u_int64_t... yes
checking for u_int64_t... (cached) yes
checking size of u_int64_t... 8
checking for uint64_t... yes
checking for uint64_t... (cached) yes
checking size of uint64_t... 8
checking for pad128_t... no
checking for upad128_t... no
checking for pid_t... yes
checking for size_t... yes
checking for size_t... (cached) yes
checking size of size_t... 8
checking for ssize_t... yes
checking for off_t... yes
checking for off_t... (cached) yes
checking size of off_t... 8
checking for mode_t... yes
checking for fd_mask... yes
checking for working alloca.h... yes
checking for alloca... yes
checking for socklen_t... yes
yes
checking sys/capability.h usability... yes
checking sys/capability.h presence... yes
checking for sys/capability.h... yes
checking for operational libcap2... (cached) dummy
checking for mtyp_t... no
checking for main in -lnsl... yes
checking for main in -lsocket... no
checking for unix domain sockets... yes
checking for main in -lgnumalloc... no
checking for main in -lmalloc... no
checking for main in -lbsd... yes
checking for main in -lregex... no
checking for gethostbyname in -lbind... no
checking for main in -lresolv... ye

Re: assert(e->mem_status == NOT_IN_MEMORY) versus TCP_MEM_HIT.

2009-09-26 Thread Amos Jeffries

Henrik Nordstrom wrote:

lör 2009-09-26 klockan 18:37 +1200 skrev Amos Jeffries:


Something seems a bit weird to me there...

(c) being harmless race condition?


It is harmless, the only ill effect is that the swap file is opened when
it does not need to be, just as happens if the request had arrived a few
ms before.

clients starting when an object is not fully in memory always opens the
disk object to be sure they can get the whole response, even if it most
times do not need to read anything from that on-disk object.

Surely its only harmless if we do (b) by changing the assert to a 
self-fix action?


The self-fix is already there in that the actual data will all be copied
from memory. It's just that not all data was in memory when the request
started (store client created) but then when doCopy was called the first
time it was.

Or at least that's my assumption on what has happened. To know for sure
the object needs to be analyzed extracting expected size and min & max
in-memory to rule out that it's not an object that has got marked
erroneous as in-memory. But I am pretty sure my guess is right.

Regards
Henrik



Ah, okay gotcha.
So...
 (c) for people needing a quick patch.
 (b) to be committed (to meet 3.2 performance goals, saving uselsss 
disk operations, etc etc).


Amos
--
Please be using
  Current Stable Squid 2.7.STABLE7 or 3.0.STABLE19
  Current Beta Squid 3.1.0.13


Re: Squid 3.1 kerb auth helper

2009-09-26 Thread Amos Jeffries

Henrik Nordstrom wrote:

lör 2009-09-26 klockan 11:43 +0100 skrev Markus Moeller:
Is this a real issue or "just" to be compliant with debian rules ?  Can you 
give me more details ?


It's the same issue I had with squid_kerb_auth when trying to package
3.1 for Fedora and which you helped to get fixed.

http://www.squid-cache.org/Versions/v3/3.1/changesets/b9719.patch


Amos, please merge at least 1 and 10002 and roll a new 3.1 release
when possible. 3.1.0.13 is just not cutting it for distro packaging, and
the amount of patches needed to get a reasonable 3.1 is rather long
now...



Aye, 3.1.0.14 is due out today...

The mention from Luigi was the last bit I wanted to see if we had a fix 
for. Since it seems already fixed and tested all the better.



[elided patches already on 3.1 and waiting]



Patches which may bite some packagers depending on compilers and enabled
Squid features:




Bug #2777: Don't know how to make target `-lrt' on OpenSolaris (not yet
in 3.1)



Other patches I ranked as critical for Fedora but unrelated to packaging





Bug #2773: Segfault in RFC2069 Digest authantication  (not yet in 3.1)


Regards
Henrik



Will do. Thanks.

Amos
--
Please be using
  Current Stable Squid 2.7.STABLE7 or 3.0.STABLE19
  Current Beta Squid 3.1.0.13


Build failed in Hudson: 3.HEAD-amd64-CentOS-5.3 #117

2009-09-26 Thread noc
See 

Changes:

[Henrik Nordstrom ] Make ESI parser modules expat 
and libxml2 dependent on their libraries

The ESI parser system is actually pluggable. There is no reason we should
require expat and libxml2. Just build what works.

Todo: Add an option to force the desired parsers.

[Amos Jeffries ] Author: Adrian Chadd 

A tproxy cache cluster (eg behind WCCPv2) can't peer.

The issue stems from the forwarding logic creating source address spoofed
sockets to destinations that are inside the cluster. Since the WCCPv2
router won't redirect packets with an origin of the proxy MAC (at least for
L2 peering), source spoofed packets go out and are routed normally. The
packets back from the destination peer have a remote end of the spoofed IP,
and are instead sent to teh original client rather than the proxy.

The forwarding logic needs to be taught to optionally enable tproxy source
spoofing on connections based on a peer flag.

Just for completeness - tproxy'ed connections to a upstream or peer proxy
which is -outside- of the WCCPv2 tproxy cluster work fine.

--
[...truncated 484 lines...]
checking varargs.h usability... no
checking varargs.h presence... no
checking for varargs.h... no
checking byteswap.h usability... yes
checking byteswap.h presence... yes
checking for byteswap.h... yes
checking glib.h usability... no
checking glib.h presence... no
checking for glib.h... no
checking for stdint.h... (cached) yes
checking for inttypes.h... (cached) yes
checking for grp.h... (cached) yes
checking db.h usability... yes
checking db.h presence... yes
checking for db.h... yes
checking db_185.h usability... yes
checking db_185.h presence... yes
checking for db_185.h... yes
checking for linux/netfilter_ipv4.h... yes
checking for linux/netfilter_ipv4/ip_tproxy.h... no
checking for net/if.h... yes
checking for netinet/if_ether.h... yes
checking for netinet/icmp6.h... yes
checking for netinet/in.h... (cached) yes
checking for netinet/ip.h... yes
checking for netinet/ip6.h... yes
checking for netinet/ip_compat.h... no
checking for netinet/ip_fil_compat.h... (cached) no
checking for netinet/ip_fil.h... no
checking for netinet/ip_icmp.h... yes
checking for netinet/ipl.h... no
checking for netinet/ip_nat.h... no
checking for net/pf/pfvar.h... no
checking for net/pfvar.h... no
checking for sys/mount.h... yes
checking for resolv.h... yes
checking libxml/parser.h usability... no
checking libxml/parser.h presence... no
checking for libxml/parser.h... no
checking libxml/parser.h usability... yes
checking libxml/parser.h presence... yes
checking for libxml/parser.h... yes
checking for an ANSI C-conforming const... yes
checking whether byte ordering is bigendian... no
checking whether struct tm is in sys/time.h or time.h... time.h
checking for struct tm.tm_gmtoff... yes
checking for struct mallinfo... yes
checking for struct mallinfo.mxfast... no
checking for struct rusage... yes
checking for struct iphdr.ip_hl... yes
checking for void *... yes
checking size of void *... 8
checking for int8_t... yes
checking for int8_t... (cached) yes
checking size of int8_t... 1
checking for char... yes
checking for char... (cached) yes
checking size of char... 1
checking for u_int8_t... yes
checking for u_int8_t... (cached) yes
checking size of u_int8_t... 1
checking for uint8_t... yes
checking for uint8_t... (cached) yes
checking size of uint8_t... 1
checking for int16_t... yes
checking for int16_t... (cached) yes
checking size of int16_t... 2
checking for short... yes
checking for short... (cached) yes
checking size of short... 2
checking for int... yes
checking for int... (cached) yes
checking size of int... 4
checking for u_int16_t... yes
checking for u_int16_t... (cached) yes
checking size of u_int16_t... 2
checking for uint16_t... yes
checking for uint16_t... (cached) yes
checking size of uint16_t... 2
checking for int32_t... yes
checking for int32_t... (cached) yes
checking size of int32_t... 4
checking for long... yes
checking for long... (cached) yes
checking size of long... 8
checking for u_int32_t... yes
checking for u_int32_t... (cached) yes
checking size of u_int32_t... 4
checking for uint32_t... yes
checking for uint32_t... (cached) yes
checking size of uint32_t... 4
checking for int64_t... yes
checking for int64_t... (cached) yes
checking size of int64_t... 8
checking for __int64... no
checking for long long... yes
checking for long long... (cached) yes
checking size of long long... 8
checking for u_int64_t... yes
checking for u_int64_t... (cached) yes
checking size of u_int64_t... 8
checking for uint64_t... yes
checking for uint64_t... (cached) yes
checking size of uint64_t... 8
checking for pad128_t... no
checking for upad128_t... no
checking for pid_t... yes
checking for size_t... yes
checking for size_t... (cached) yes
checking size of size_t... 8
checking for ssize_t... yes
checking for off_t... yes
checking for off_t.

Hudson build is back to normal: 2.HEAD-i386-FreeBSD-6.4 #233

2009-09-26 Thread noc
See 




Build failed in Hudson: 2.HEAD-i386-FreeBSD-6.4 #232

2009-09-26 Thread noc
See 

--
Started by upstream project "2.HEAD-amd64-CentOS-5.3" build number 237
Building remotely on squid-cache.org
cvs checkout: warning: failed to open /home/robertc/.cvspass for reading: No 
such file or directory
$ computing changelog
cvs log: warning: failed to open /home/robertc/.cvspass for reading: No such 
file or directory
Fatal error, aborting.
anoncvs_squid: no such system user
ERROR: cvs exited with error code 1
Command line was [Executing 'cvs' with arguments:
'-d:pserver:anonym...@cvs.devel.squid-cache.org:/cvsroot/squid'
'log'
'-S'
'-d'
'>=2009-09-25'

The ' characters around the executable and arguments are
not part of the command.
] in 
Input text:
START==>null<==END
Working directory is 

cvs exited with error code 1
Command line was [Executing 'cvs' with arguments:
'-d:pserver:anonym...@cvs.devel.squid-cache.org:/cvsroot/squid'
'log'
'-S'
'-d'
'>=2009-09-25'

The ' characters around the executable and arguments are
not part of the command.
] in 
Input text:
START==>null<==END
at 
hudson.org.apache.tools.ant.taskdefs.AbstractCvsTask.runCommand(AbstractCvsTask.java:387)
at 
hudson.org.apache.tools.ant.taskdefs.AbstractCvsTask.execute(AbstractCvsTask.java:457)
at 
hudson.org.apache.tools.ant.taskdefs.cvslib.ChangeLogTask.execute(ChangeLogTask.java:293)
at hudson.scm.CVSSCM$4.invoke(CVSSCM.java:894)
at hudson.scm.CVSSCM$4.invoke(CVSSCM.java:829)
at hudson.FilePath$FileCallableWrapper.call(FilePath.java:2032)
at hudson.remoting.UserRequest.perform(UserRequest.java:103)
at hudson.remoting.UserRequest.perform(UserRequest.java:47)
at hudson.remoting.Request$2.run(Request.java:236)
at 
java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:441)
at java.util.concurrent.FutureTask$Sync.innerRun(FutureTask.java:303)
at java.util.concurrent.FutureTask.run(FutureTask.java:138)
at 
java.util.concurrent.ThreadPoolExecutor$Worker.runTask(ThreadPoolExecutor.java:885)
at 
java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:907)
at hudson.remoting.Engine$1$1.run(Engine.java:54)
at java.lang.Thread.run(Thread.java:619)



Re: Squid 3.1 kerb auth helper

2009-09-26 Thread Henrik Nordstrom
lör 2009-09-26 klockan 11:43 +0100 skrev Markus Moeller:
> Is this a real issue or "just" to be compliant with debian rules ?  Can you 
> give me more details ?

It's the same issue I had with squid_kerb_auth when trying to package
3.1 for Fedora and which you helped to get fixed.

http://www.squid-cache.org/Versions/v3/3.1/changesets/b9719.patch


Amos, please merge at least 1 and 10002 and roll a new 3.1 release
when possible. 3.1.0.13 is just not cutting it for distro packaging, and
the amount of patches needed to get a reasonable 3.1 is rather long
now...

Required packaging patches:

Correct squid_kerb_auth compile/link flags to avoid bad runpath settings etc


Cleanup automake-foo a bit in errors/ (fixes lang symlinks when using
make install DESTDIR=...)


Install error page templates properly. (correction to the above)



Patches which may bite some packagers depending on compilers and enabled
Squid features:

Better const-correctness on FTP login parse (newer GCC barfing)


Fixup libxml2 include magics, was failing when a configure cache was
used (ESI related)


Bug #2734: fix compile errors from CBDATA_CLASS2()


Make ESI behave reasonable when built but not used


Bug #2777: Don't know how to make target `-lrt' on OpenSolaris (not yet
in 3.1)



Other patches I ranked as critical for Fedora but unrelated to packaging

Bug #2718: FTP sends EPSV2 on ipv4 connection


Bug #2541: Hang in 100% CPU loop while extacting header details using a
delimiter other than comma


Bug #2745: Invalid response error on small reads


Bug #2624: Invalid response for IMS request


Bug #2773: Segfault in RFC2069 Digest authantication  (not yet in 3.1)


Regards
Henrik



Re: assert(e->mem_status == NOT_IN_MEMORY) versus TCP_MEM_HIT.

2009-09-26 Thread Henrik Nordstrom
lör 2009-09-26 klockan 18:37 +1200 skrev Amos Jeffries:

> Something seems a bit weird to me there...
> 
> (c) being harmless race condition?

It is harmless, the only ill effect is that the swap file is opened when
it does not need to be, just as happens if the request had arrived a few
ms before.

clients starting when an object is not fully in memory always opens the
disk object to be sure they can get the whole response, even if it most
times do not need to read anything from that on-disk object.

> Surely its only harmless if we do (b) by changing the assert to a 
> self-fix action?

The self-fix is already there in that the actual data will all be copied
from memory. It's just that not all data was in memory when the request
started (store client created) but then when doCopy was called the first
time it was.

Or at least that's my assumption on what has happened. To know for sure
the object needs to be analyzed extracting expected size and min & max
in-memory to rule out that it's not an object that has got marked
erroneous as in-memory. But I am pretty sure my guess is right.

Regards
Henrik



Re: Squid 3.1 kerb auth helper

2009-09-26 Thread Markus Moeller
If I run 
./configure --enable-negotiate-auth-helpers=squid_kerb_auth --prefix=/opt/squid-3.1


I don't get an rpath. I use debian etch.

Markus

make[3]: Entering directory 
`/home/markus/squid-3.1.0.13-20090926/helpers/negotiate_auth/squid_kerb_auth'

make  all-recursive
make[4]: Entering directory 
`/home/markus/squid-3.1.0.13-20090926/helpers/negotiate_auth/squid_kerb_auth'
make[5]: Entering directory 
`/home/markus/squid-3.1.0.13-20090926/helpers/negotiate_auth/squid_kerb_auth'
gcc -DHAVE_CONFIG_H -I. -Ispnegohelp   -I../../../ -I../../../include/ -I/home/markus/squid-3.1.0.13-20090926/include 
-I/home/markus/squid-3.1.0.13-20090926/src -I/home/markus/squid-3.1.0.13-20090926 
 -g -O2 -Wall -Wextra -Werror -Wcomment -Wpointer-arith -Wcast-align -Wwrite-strings 
-Wstrict-prototypes -Wmissing-prototypes -Wmissing-declarations -Wdeclaration-after-statement 
-Wshadow -MT squid_kerb_auth.o -MD -MP -MF .deps/squid_kerb_auth.Tpo -c -o 
squid_kerb_auth.o squid_kerb_auth.c

mv -f .deps/squid_kerb_auth.Tpo .deps/squid_kerb_auth.Po
gcc -DHAVE_CONFIG_H -I. -Ispnegohelp   -I../../../ -I../../../include/ -I/home/markus/squid-3.1.0.13-20090926/include 
-I/home/markus/squid-3.1.0.13-20090926/src -I/home/markus/squid-3.1.0.13-20090926 
 -g -O2 -Wall -Wextra -Werror -Wcomment -Wpointer-arith -Wcast-align -Wwrite-strings 
-Wstrict-prototypes -Wmissing-prototypes -Wmissing-declarations -Wdeclaration-after-statement 
-Wshadow -MT base64.o -MD -MP -MF .deps/base64.Tpo -c -o base64.o base64.c

mv -f .deps/base64.Tpo .deps/base64.Po
gcc -DHAVE_CONFIG_H -I. -Ispnegohelp   -I../../../ -I../../../include/ -I/home/markus/squid-3.1.0.13-20090926/include 
-I/home/markus/squid-3.1.0.13-20090926/src -I/home/markus/squid-3.1.0.13-20090926 
 -g -O2 -Wall -Wextra -Werror -Wcomment -Wpointer-arith -Wcast-align -Wwrite-strings 
-Wstrict-prototypes -Wmissing-prototypes -Wmissing-declarations -Wdeclaration-after-statement 
-Wshadow -MT derparse.o -MD -MP -MF .deps/derparse.Tpo -c -o derparse.o 
`test -f 'spnegohelp/derparse.c' || echo './'`spnegohelp/derparse.c

mv -f .deps/derparse.Tpo .deps/derparse.Po
gcc -DHAVE_CONFIG_H -I. -Ispnegohelp   -I../../../ -I../../../include/ -I/home/markus/squid-3.1.0.13-20090926/include 
-I/home/markus/squid-3.1.0.13-20090926/src -I/home/markus/squid-3.1.0.13-20090926 
 -g -O2 -Wall -Wextra -Werror -Wcomment -Wpointer-arith -Wcast-align -Wwrite-strings 
-Wstrict-prototypes -Wmissing-prototypes -Wmissing-declarations -Wdeclaration-after-statement 
-Wshadow -MT spnego.o -MD -MP -MF .deps/spnego.Tpo -c -o spnego.o `test -f 
'spnegohelp/spnego.c' || echo './'`spnegohelp/spnego.c

mv -f .deps/spnego.Tpo .deps/spnego.Po
gcc -DHAVE_CONFIG_H -I. -Ispnegohelp   -I../../../ -I../../../include/ -I/home/markus/squid-3.1.0.13-20090926/include 
-I/home/markus/squid-3.1.0.13-20090926/src -I/home/markus/squid-3.1.0.13-20090926 
 -g -O2 -Wall -Wextra -Werror -Wcomment -Wpointer-arith -Wcast-align -Wwrite-strings 
-Wstrict-prototypes -Wmissing-prototypes -Wmissing-declarations -Wdeclaration-after-statement 
-Wshadow -MT spnegohelp.o -MD -MP -MF .deps/spnegohelp.Tpo -c -o 
spnegohelp.o `test -f 'spnegohelp/spnegohelp.c' || echo 
'./'`spnegohelp/spnegohelp.c

mv -f .deps/spnegohelp.Tpo .deps/spnegohelp.Po
gcc -DHAVE_CONFIG_H -I. -Ispnegohelp   -I../../../ -I../../../include/ -I/home/markus/squid-3.1.0.13-20090926/include 
-I/home/markus/squid-3.1.0.13-20090926/src -I/home/markus/squid-3.1.0.13-20090926 
 -g -O2 -Wall -Wextra -Werror -Wcomment -Wpointer-arith -Wcast-align -Wwrite-strings 
-Wstrict-prototypes -Wmissing-prototypes -Wmissing-declarations -Wdeclaration-after-statement 
-Wshadow -MT spnegoparse.o -MD -MP -MF .deps/spnegoparse.Tpo -c -o 
spnegoparse.o `test -f 'spnegohelp/spnegoparse.c' || echo 
'./'`spnegohelp/spnegoparse.c

mv -f .deps/spnegoparse.Tpo .deps/spnegoparse.Po
gcc  -g -O2 -Wall -Wextra -Werror -Wcomment -Wpointer-arith -Wcast-align -Wwrite-strings 
-Wstrict-prototypes -Wmissing-prototypes -Wmissing-declarations -Wdeclaration-after-statement 
-Wshadow  -L/usr/lib -lgssapi_krb5 -lkrb5 -lk5crypto -lcom_err -lkrb5support 
-lresolv -L../../../lib -o squid_kerb_auth squid_kerb_auth.o base64.o 
derparse.o spnego.o spnegohelp.o spnegoparse.o  -lmiscutil -lm
gcc -DHAVE_CONFIG_H -I. -Ispnegohelp   -I../../../ -I../../../include/ -I/home/markus/squid-3.1.0.13-20090926/include 
-I/home/markus/squid-3.1.0.13-20090926/src -I/home/markus/squid-3.1.0.13-20090926 
 -g -O2 -Wall -Wextra -Werror -Wcomment -Wpointer-arith -Wcast-align -Wwrite-strings 
-Wstrict-prototypes -Wmissing-prototypes -Wmissing-declarations -Wdeclaration-after-statement 
-Wshadow -MT squid_kerb_auth_test.o -MD -MP -MF 
.deps/squid_kerb_auth_test.Tpo -c -o squid_kerb_auth_test.o 
squid_kerb_auth_test.c

mv -f .deps/squid_kerb_auth_test.Tpo .deps/squid_kerb_auth_test.Po
gcc  -g -O2 -Wall -Wextra -Werror -Wcomment -Wpointer-arith -Wcast-align -Wwrite-st

Re: Squid 3.1 kerb auth helper

2009-09-26 Thread Markus Moeller


Is this a real issue or "just" to be compliant with debian rules ?  Can you 
give me more details ?


Thank you
Markus

- Original Message - 
From: "Amos Jeffries" 
To: "Markus Moeller" ; "Squid Developers" 
; "Luigi Gangitano" 

Sent: Saturday, September 26, 2009 2:11 AM
Subject: Squid 3.1 kerb auth helper



Hi Markus,
  We've hit another issue with the new 3.1 helper. This time on Debian...

Luigi Gangitano wrote:
>
> At last, I've a small issue with negotiate_auth/squid_kerb_auth, which
> set RPATH in binaries. configure script adds '-Wl,-R' to the linker
> under squid. This is an issue for debian (see
> http://wiki.debian.org/RpathIssue). Do you think this can be fixed? I
> can make a debian patch if needed.

Amos
--
Please be using
  Current Stable Squid 2.7.STABLE7 or 3.0.STABLE19
  Current Beta Squid 3.1.0.13






Re: assert(e->mem_status == NOT_IN_MEMORY) versus TCP_MEM_HIT.

2009-09-26 Thread Amos Jeffries

Henrik Nordstrom wrote:

fre 2009-09-25 klockan 23:40 +1200 skrev Amos Jeffries:

This is the current blocker on trunk...

Attempting to swap-in an object which is already in memory and on disk.


Right, there is a timing race uncovered by the swapin changes.

1. store client created and type set to disk client.

[some time may pass]

2. First doCopy call triggers the start of the swapin.


I see two ways of fixing this:

a) Move the swapin start to 1.

b) Make store clients automatically switch from disk to mem type if the
object is found to be already swapped in in doCopy

or alternatively

c) Remove the assert. It's harmless.


Something seems a bit weird to me there...

(c) being harmless race condition?

Surely its only harmless if we do (b) by changing the assert to a 
self-fix action?


If you are right, I'd prefer (b) anyway.

Amos
--
Please be using
  Current Stable Squid 2.7.STABLE7 or 3.0.STABLE19
  Current Beta Squid 3.1.0.13