openssh unix dev - Aug 2001 - [PATCH]: Drop Cygwin workaround

Hi,

the Cygwin specific workaround to sleep a second in daemon() isn't
needed anymore. We can eliminate it from the OpenSSH sources.

Index: daemon.c
==================================================================RCS file:
/cvs/openssh_cvs/openbsd-compat/daemon.c,v
retrieving revision 1.1
diff -u -p -r1.1 daemon.c
--- daemon.c	2001/01/31 21:52:03	1.1
+++ daemon.c	2001/08/30 08:41:39
@@ -51,13 +51,6 @@ daemon(nochdir, noclose)
 	case 0:
 		break;
 	default:
-#ifdef HAVE_CYGWIN
-		/*
-		 * This sleep avoids a race condition which kills the
-		 * child process if parent is started by a NT/W2K service.
-		 */
-		sleep(1);
-#endif
 		_exit(0);
 	}
 

Corinna

-- 
Corinna Vinschen
Cygwin Developer
Red Hat, Inc.
mailto:vinschen at redhat.com

Anybody here with write access to the CVS repository?

Corinna

On Thu, Aug 30, 2001 at 10:43:48AM +0200, Corinna Vinschen
wrote:
> Hi,
> 
> the Cygwin specific workaround to sleep a second in daemon() isn't
> needed anymore. We can eliminate it from the OpenSSH sources.
> 
> Index: daemon.c
> ==================================================================> RCS
file: /cvs/openssh_cvs/openbsd-compat/daemon.c,v
> retrieving revision 1.1
> diff -u -p -r1.1 daemon.c
> --- daemon.c	2001/01/31 21:52:03	1.1
> +++ daemon.c	2001/08/30 08:41:39
> @@ -51,13 +51,6 @@ daemon(nochdir, noclose)
>  	case 0:
>  		break;
>  	default:
> -#ifdef HAVE_CYGWIN
> -		/*
> -		 * This sleep avoids a race condition which kills the
> -		 * child process if parent is started by a NT/W2K service.
> -		 */
> -		sleep(1);
> -#endif
>  		_exit(0);
>  	}
-- 
Corinna Vinschen
Cygwin Developer
Red Hat, Inc.
mailto:vinschen at redhat.com

On Thu, 30 Aug 2001, Corinna Vinschen wrote:
> Hi,
>
> the Cygwin specific workaround to sleep a second in daemon() isn't
> needed anymore. We can eliminate it from the OpenSSH sources.
Is it not needed because of changes to the OpenSSH code or
changes to Cygwin?
>
> Index: daemon.c
> ==================================================================> RCS
file: /cvs/openssh_cvs/openbsd-compat/daemon.c,v
> retrieving revision 1.1
> diff -u -p -r1.1 daemon.c
> --- daemon.c	2001/01/31 21:52:03	1.1
> +++ daemon.c	2001/08/30 08:41:39
> @@ -51,13 +51,6 @@ daemon(nochdir, noclose)
>  	case 0:
>  		break;
>  	default:
> -#ifdef HAVE_CYGWIN
> -		/*
> -		 * This sleep avoids a race condition which kills the
> -		 * child process if parent is started by a NT/W2K service.
> -		 */
> -		sleep(1);
> -#endif
>  		_exit(0);
>  	}
>
>
> Corinna
>
>
-- 
Tim Rice				Multitalents	(707) 887-1469
tim at multitalents.net