Re: [PATCH] cygrunsrv --query <service>

[Corinna Vinschen <cygwin at cygwin dot com>]

On Fri, Mar 08, 2002 at 03:04:38PM -0500, Mike Gerdts wrote:
> The following patch implements the options -Q and --query to determine
> if a service is installed and if so whether it is up and what controls
> it will accept.

Cool patch!  Thanks, I've applied it to the CVS repository and I'm
going to upload a new version of cygrunsrv soon.

> If there is a more appropriate place to send this patch, please let me
> know.

Yes, you could send that stuff to cygwin-apps@cygwin.com.  If you
don't want to subscribe to YA mailing list, just keep it here.

A side note though, related to your ChangeLog entry.

First, it's not correct.  Bullets only for a new file entry; don't repeat
long texts if avoidable.  Better:


  2002-03-08  Mike Gerdts <Michael.Gerdts@alcatel.com>

	  * utils.h (reason_t): Add code for --query errors.
	  * utils.cc (reason_list): Add error string for --query errors.
	  (usage): Add help text for --query parameter.
	  * cygrunsrv.cc (longopts): Add --query parameter.
	  (opts): Add -Q option.
	  (main): Add handling --query parameter.
	  (serviceTypeToString): New function to support --query parameter.
	  (serviceStateToString): Ditto.
	  (serviceStateToString): Ditto.
	  (controlsToString): Ditto.
	  (query_service): Ditto.

Second, please don't send ChangeLog entries as part of the diff.
Send them always separately clear text.  You never know if the
ChangeLog hasn't changed in the meantime and diffs to ChangeLogs
never apply cleanly then.

Thanks,
Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Developer                                mailto:cygwin@cygwin.com
Red Hat, Inc.

--
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple
Bug reporting:         http://cygwin.com/bugs.html
Documentation:         http://cygwin.com/docs.html
FAQ:                   http://cygwin.com/faq/