Mailing-List: contact cygwin-help AT cygwin DOT com; run by ezmlm List-Subscribe: List-Archive: List-Post: List-Help: , Sender: cygwin-owner AT cygwin DOT com Mail-Followup-To: cygwin AT cygwin DOT com Delivered-To: mailing list cygwin AT cygwin DOT com Date: Thu, 19 May 2005 22:30:46 +0200 From: Corinna Vinschen To: cygwin AT cygwin DOT com Cc: cygwin-patches AT cygwin DOT com Subject: Re: [patch] several new features for cygrunsrv Message-ID: <20050519203046.GH2794@calimero.vinschen.de> Reply-To: cygwin AT cygwin DOT com Mail-Followup-To: cygwin AT cygwin DOT com, cygwin-patches AT cygwin DOT com References: <428CE837 DOT C00E288B AT dessent DOT net> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <428CE837.C00E288B@dessent.net> User-Agent: Mutt/1.4.2i 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 it will be double-quoted. I gave the -L option a quick test and to my surprise, no service got printed. The problem was apparently that I was not running /bin/cygrunsrv, but the cygrunsrv I've just built in my builddir. To recognize the services as a Cygwin service, the list function uses the full path to cygrunsrv. I'm wondering if comparing the basename wouldn't be better for debugging purposes? This is also more aligned to the -Q option which prints services also if it's not the "right" cygrunsrv asking for them. Thanks, Corinna P.S.: I'm redirecting to the cygwin ML. cygwin-patches is really for Cygwin only, not for any Cygwin applications. -- Corinna Vinschen Please, send mails regarding Cygwin to Cygwin Project Co-Leader mailto:cygwin AT cygwin DOT com Red Hat, Inc. -- Unsubscribe info: http://cygwin.com/ml/#unsubscribe-simple Problem reports: http://cygwin.com/problems.html Documentation: http://cygwin.com/docs.html FAQ: http://cygwin.com/faq/