This is the mail archive of the cygwin-apps@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]
Other format: [Raw text]

Re: AW: AW: [PATCH] setup - help and local dir command lineoptionswas"Re: Setup Command Line Options"


On Sat, 2003-11-01 at 12:16, Ralf Habacker wrote:
> Rob
> > I need more time to think about what you have written and how to
> > start with which class,
> >
> I have take some time to inspect how it could be go and have build a
> testcase to see what kind of api is needed. Please note that I am
> concentrated only to get a command line version initial running, so I've
> removed the not needed gui stuff.  Currently I can't send in a patch for
> this, because I have splitted the relating engine files into subdir and
> build a static library to allow file including checks, although the binariy
> and sources could be downloaded  here:
> 
> binary http://kde-cygwin.sf.net/snapshots/enginetest-0.0.1.exe.
> source http://kde-cygwin.sf.net/snapshots/setup_new-0.0.1.tar.bz2

Sounds like you've gone too far too fast. I don't have the time to
review a non-delta set of changes. Particularly ones that throw away a
lot of code (the gui) that -must- be retained. 

> Any comments ?

Yes. I sketched out a fairly minimal development path to generate a
commandline setup that doesn't involve forking the engine code. You
haven't commented on the approach, rather you appear to have just
ignored it completely.

Shrug.
Rob

-- 
GPG key available at: <http://members.aardvark.net.au/lifeless/keys.txt>.

Attachment: signature.asc
Description: This is a digitally signed message part


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