commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bill Barker" <wbar...@wilshire.com>
Subject Re: cvs commit: jakarta-commons/daemon/src/native/unix/native jsvc-unix.c
Date Sat, 03 Jan 2004 04:45:38 GMT

----- Original Message ----- 
From: "jean-frederic clere" <jfrederic.clere@fujitsu-siemens.com>
To: <billbarker@apache.org>
Sent: Friday, January 02, 2004 1:04 AM
Subject: Re: cvs commit: jakarta-commons/daemon/src/native/unix/native
jsvc-unix.c


> Thanks. Happy new year.
>
> Do you want to release daemon soon?

We probably should.  There are still 4 bugs outstanding for Daemon:
  23365) Probably a 'wontfix', or possibly a 'later'.  I don't have strong
opinions on this one.
  24936) I'll probably patch this one over the weekend (it's the last
procrun bug)
  23723) I don't have access to Mac OS/X, so I can't really comment.
  23368) You seem to be working on this one, so I haven't really looked at
it, but it's an Enh.

The only other question is whether or not Yoav wants to continue as the RM
(he's done a great job so far, so he has my +1 if he wants the job :).

>
> Cheers
>
> Jean-Frederic
>
>
> billbarker@apache.org wrote:
> > billbarker    2003/12/30 21:03:26
> >
> >   Modified:    daemon/src/native/unix/native jsvc-unix.c
> >   Log:
> >   Fix problems with signal handling on Solaris (possibly others).
> >
> >   Fix for Bug #24247
> >
> >   Reported By: Peter Poloha  peter.poloha@nextra.sk
> >
> >   Revision  Changes    Path
> >   1.8       +4 -4
jakarta-commons/daemon/src/native/unix/native/jsvc-unix.c
> >
> >   Index: jsvc-unix.c
> >   ===================================================================
> >   RCS file:
/home/cvs/jakarta-commons/daemon/src/native/unix/native/jsvc-unix.c,v
> >   retrieving revision 1.7
> >   retrieving revision 1.8
> >   diff -u -r1.7 -r1.8
> >   --- jsvc-unix.c 27 Sep 2003 16:49:13 -0000 1.7
> >   +++ jsvc-unix.c 31 Dec 2003 05:03:25 -0000 1.8
> >   @@ -271,10 +271,10 @@
> >    /*
> >     * Return the address of the current signal handler and set the new
one.
> >     */
> >   -static void * signal_set(int sig, void * handler) {
> >   +static void * signal_set(int sig, void * newHandler) {
> >        void *hand;
> >
> >   -    hand=signal(sig,handler);
> >   +    hand=signal(sig,newHandler);
> >    #ifdef SIG_ERR
> >        if (hand==SIG_ERR)
> >            hand=NULL;
> >   @@ -350,7 +350,7 @@
> >        /* Install signal handlers */
> >        handler_hup=signal_set(SIGHUP,handler);
> >        handler_trm=signal_set(SIGTERM,handler);
> >   -    handler_trm=signal_set(SIGINT,handler);
> >   +    handler_int=signal_set(SIGINT,handler);
> >        controlled = getpid();
> >        log_debug("Waiting for a signal to be delivered");
> >        while (!stopping) sleep(60); /* pause() not threadsafe */
> >
> >
> >
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
> > For additional commands, e-mail: commons-dev-help@jakarta.apache.org
> >
> >
>
>


Mime
View raw message