Re: [patch] several new features for cygrunsrv

2005-05-19 Thread Corinna Vinschen
Hi Brian, I really like this patch, cool stuff. However, I have two nits. First, your patch adds new options, so it should also add some wording to cygrunsrv.README. On May 19 12:25, Brian Dessent wrote: > for S in `cygrunsrv -L`; do cygrunsrv -E $S; done > > If a service name contains spaces

Re: [patch] several new features for cygrunsrv

2005-05-19 Thread Christopher Faylor
On Thu, May 19, 2005 at 12:25:43PM -0700, Brian Dessent wrote: > >(I realize this list is technically only for patches on cygwin, but cygrunsrv >is >close enough...) Actually, no it isn't. Please use the cygwin list for cygrunsrv. (nice additions to cygrunsrv, though) cgf

Re: [patch] several new features for cygrunsrv

2005-05-19 Thread Brian Dessent
Brian Dessent wrote: > -controlsToString(DWORD controls) > + char *base, *end; > + static char buf[34]; > + int used = 0, dsiz = strlen (delim); Crap, that is a mistake. buf[34] should be something more generous like 128 or 256. I had it set small to test to make sure it couldn't overflow an

[patch] several new features for cygrunsrv

2005-05-19 Thread Brian Dessent
(I realize this list is technically only for patches on cygwin, but cygrunsrv is close enough...) I've implemented some new features for cygrunsrv that I think might be useful to people. Examples follow. A new command "-L" / "--list" can be specified that will list all installed cygrunsrv servi

Re: [Patch]: mkdir -p and network drives

2005-05-19 Thread Eric Blake
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Wrong list. Redirecting. According to Vance Turner on 5/18/2005 11:06 PM: > I usually don't write you guys, I follow the thread to see how development > is going. > > Just a note. The ls command is't quite right. > > Ls -lRC wil not recursively li

RE: [Patch]: mkdir -p and network drives

2005-05-19 Thread Dave Korn
Original Message >From: Christopher Faylor >Sent: 19 May 2005 06:13 > On Wed, May 18, 2005 at 10:09:35PM -0700, Vance Turner wrote: >> Additional note >> >> ls -lRC - not working >> ls -RCl - working >> >> If you point out the source I will fix it. > > 1) This is not a bug reporting lis