This is the mail archive of the cygwin-patches@cygwin.com 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]

Re: *.cwp file recognistion patch for setup


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


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