You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@apr.apache.org by Sander Striker <st...@apache.org> on 2003/06/02 16:50:20 UTC
RE: [PATCH] Multi-threaded svnserve
[cross posted to dev@apr]
> From: rbb@rkbloom.net [mailto:rbb@rkbloom.net]
> Sent: Monday, June 02, 2003 4:54 PM
> On Mon, 2 Jun 2003, Sander Striker wrote:
>
> > > From: cmpilato@collab.net [mailto:cmpilato@collab.net]
> > > Sent: Monday, June 02, 2003 4:05 PM
> >
> > > I still don't have SIGCHLD (should that be APR_SIGCHLD or
> > > something?). But the apr_proc_fork() error is gone.
> >
> > Hmmm:
> >
> > include/arch/win32/apr_private.h:139:#define SIGCHLD 17
> >
> > I guess some includes are off.
>
> No they aren't. Program shouldn't be incuding apr_private.h, so you can't
> expect SIGCHLD to be defined. Either a public APR header file redefines
> it to APR_SIGCHLD, or it is only meant to be used inside APR.
I know. I was assuming that the public header where this happens wasn't
being included. However, some more research shows that SIGCHLD doesn't
appear anywhere in the public headers. I expected something in apr_signal.h,
but nothing there.
Sander
Re: [PATCH] Multi-threaded svnserve
Posted by cm...@collab.net.
"Sander Striker" <st...@apache.org> writes:
> > No they aren't. Program shouldn't be incuding apr_private.h, so you can't
> > expect SIGCHLD to be defined. Either a public APR header file redefines
> > it to APR_SIGCHLD, or it is only meant to be used inside APR.
>
> I know. I was assuming that the public header where this happens wasn't
> being included. However, some more research shows that SIGCHLD doesn't
> appear anywhere in the public headers. I expected something in apr_signal.h,
> but nothing there.
Yeah, I expected the same (and found the same ... or lack of the same
... or something).
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Re: [PATCH] Multi-threaded svnserve
Posted by cm...@collab.net.
"Sander Striker" <st...@apache.org> writes:
> > No they aren't. Program shouldn't be incuding apr_private.h, so you can't
> > expect SIGCHLD to be defined. Either a public APR header file redefines
> > it to APR_SIGCHLD, or it is only meant to be used inside APR.
>
> I know. I was assuming that the public header where this happens wasn't
> being included. However, some more research shows that SIGCHLD doesn't
> appear anywhere in the public headers. I expected something in apr_signal.h,
> but nothing there.
Yeah, I expected the same (and found the same ... or lack of the same
... or something).