This is the mail archive of the cygwin-apps mailing list for the Cygwin project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH] Multiple --site options

From: Jon TURNEY <jon dot turney at dronecode dot org dot uk>
To: cygwin-apps at cygwin dot com
Date: Wed, 22 Feb 2012 17:53:19 +0000
Subject: Re: [PATCH] Multiple --site options
References: <4DE36335.1010404@ludens.elte.hu> <4E297C1E.9020200@dronecode.org.uk> <4E2DB97B.2060107@ludens.elte.hu> <4F44CED8.2080303@dronecode.org.uk> <20120222153401.GA11124@ednor.casa.cgf.cx>

On 22/02/12 15:34, Christopher Faylor wrote:

On Wed, Feb 22, 2012 at 11:17:44AM +0000, Jon TURNEY wrote:

Looking at this again, I withdrawn my comment on putting common code in the
Option class.  This patch matches the existing design of having no code in the
Option class.

So can we apply this? It's correct, and it works.


Do you want to do it Jon?  I haven't looked at it in detail (i.e., at
all) but if you think it's ok, that works for me.

Sure, I'll do it.

References:
- Re: [PATCH] Multiple --site options
  - From: Jon TURNEY
- Re: [PATCH] Multiple --site options
  - From: Christopher Faylor

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]