Re: [PATCH] We have sysvsem on OpenBSD

2002-07-17 Thread Aaron Bannert
On Wed, Jul 17, 2002 at 07:23:08AM -0700, Ryan Bloom wrote: > > Why isn't this being detected by autoconf? SysV semaphore support > > isn't perfect yet and has some problems. > > Because this is Apache 1.3 which doesn't use autoconf. :-) This is my brain -= sleep. :) -aaron

RE: [PATCH] We have sysvsem on OpenBSD

2002-07-17 Thread Ryan Bloom
> From: Aaron Bannert [mailto:[EMAIL PROTECTED]] > > On Wed, Jul 17, 2002 at 10:31:44AM -0400, Jeff Trawick wrote: > > does everybody agree that this is preferable? > > Why isn't this being detected by autoconf? SysV semaphore support > isn't perfect yet and has some problems. Because this is

Re: [PATCH] We have sysvsem on OpenBSD

2002-07-17 Thread Aaron Bannert
On Wed, Jul 17, 2002 at 10:31:44AM -0400, Jeff Trawick wrote: > does everybody agree that this is preferable? Why isn't this being detected by autoconf? SysV semaphore support isn't perfect yet and has some problems. -aaron > Index: src/include/ap_config.h > ===

Re: [PATCH] We have sysvsem on OpenBSD

2002-07-17 Thread Henning Brauer
On Wed, Jul 17, 2002 at 10:31:44AM -0400, Jeff Trawick wrote: > does everybody agree that this is preferable? > > Index: src/include/ap_config.h > === > RCS file: /home/cvs/apache-1.3/src/include/ap_config.h,v > retrieving revision 1

Re: [PATCH] We have sysvsem on OpenBSD

2002-07-17 Thread Jim Jagielski
Jeff Trawick wrote: > > does everybody agree that this is preferable? > > Index: src/include/ap_config.h > === > RCS file: /home/cvs/apache-1.3/src/include/ap_config.h,v > retrieving revision 1.323 > diff -u -r1.323 ap_config.h > --

Re: [PATCH] We have sysvsem on OpenBSD

2002-07-17 Thread Jeff Trawick
Henning Brauer <[EMAIL PROTECTED]> writes: > On Wed, Jul 17, 2002 at 08:42:33AM -0400, Jeff Trawick wrote: > > I would have thought you would add > > > > #if defined __OpenBSD__ > > #define HAVE_SYSVSEM_SERIALIZED_ACCEPT > > #endif > > > > since you have flock too. > > > > If you want to

Re: [PATCH] We have sysvsem on OpenBSD

2002-07-17 Thread Jim Jagielski
Henning Brauer wrote: > > > Why the "#else" part? Does OpenBSD lack flock() or is it because of > > the new 'chroot' that your version does, but the .org doesn't. If the > > latter, I'd prefer allowing both with the official ASF code. > > oversight on our part. > I just commited a fix. > > floc

Re: [PATCH] We have sysvsem on OpenBSD

2002-07-17 Thread Henning Brauer
On Wed, Jul 17, 2002 at 03:22:54PM +0200, Henning Brauer wrote: > flock works with out chroot, but we don't want www-writeable files inside > the chroot, so we default to sysvsem. flock works with our chroot, that is.

Re: [PATCH] We have sysvsem on OpenBSD

2002-07-17 Thread Henning Brauer
On Wed, Jul 17, 2002 at 09:18:34AM -0400, Jim Jagielski wrote: > Henning Brauer wrote: > > > > sorry if this appears twice, first one seems not to have made it. > > > > we have sysvsem on OpenBSD and applied the following patch to ap_config.h > > in our tree - would be nice to have it in the ap

Re: [PATCH] We have sysvsem on OpenBSD

2002-07-17 Thread Jim Jagielski
Henning Brauer wrote: > > sorry if this appears twice, first one seems not to have made it. > > we have sysvsem on OpenBSD and applied the following patch to ap_config.h > in our tree - would be nice to have it in the apache.org tree as well. this > is for apache 1.3. > > Index: src/include/ap

Re: [PATCH] We have sysvsem on OpenBSD

2002-07-17 Thread Henning Brauer
On Wed, Jul 17, 2002 at 08:42:33AM -0400, Jeff Trawick wrote: > I would have thought you would add > > #if defined __OpenBSD__ > #define HAVE_SYSVSEM_SERIALIZED_ACCEPT > #endif > > since you have flock too. > > If you want to force sysvsem be the default, add > > #define USE_SYSVSEM_SE

Re: [PATCH] We have sysvsem on OpenBSD

2002-07-17 Thread Jeff Trawick
Henning Brauer <[EMAIL PROTECTED]> writes: > we have sysvsem on OpenBSD and applied the following patch to ap_config.h > in our tree - would be nice to have it in the apache.org tree as well. this > is for apache 1.3. > > Index: src/include/ap_config.h >