This is the mail archive of the
cygwin-patches@cygwin.com
mailing list for the Cygwin project.
Re: *.cwp file recognistion patch for setup
- To: cygwin-patches at cygwin dot com
- Subject: Re: *.cwp file recognistion patch for setup
- From: Christopher Faylor <cgf at redhat dot com>
- Date: Tue, 11 Sep 2001 10:50:17 -0400
- References: <999956948.9501.93.camel@lifelesswks> <3B9DD027.7C8A87A8@etr-usa.com> <1000215434.7293.235.camel@lifelesswks>
- Reply-To: cygwin-patches at cygwin dot com
On Tue, Sep 11, 2001 at 11:37:13PM +1000, Robert Collins wrote:
>On Tue, 2001-09-11 at 18:49, Warren Young wrote:
>> > 1) just _ignore_ the file extention.
>>
>> I agree. My original patch was a trivial patch to add this
>> functionality -- this was one of the design goals, if you saw the
>> argument in the "press for cygwin" thread. :) Now it seems the Cygwin
>> team wants it done right, and not just "done". Okay, I can do that
>> instead, now that the point has been made. :)
>
>Yes, I recall Chris's comment about non-trivial. This is really the
>point - "we" can put a trivial patch into setup, but peer review means
>that "we" will all pull it to pieces. So a non-trivial, done the "Right
>Way" patch is needed...
Yes. My non-trivial comment had a dual meaning.
Also, I said repeatedly in the thread that started this discussion that
we didn't want to do things the hacky way. That is the WHOLE REASON for
this continued thread, in fact.
cgf