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: [PATCH] Re: [setup] Inaccurate message: See /setup.log.full


On Sun, 2003-02-02 at 08:44, Max Bowsher wrote:


> > No it's not, because the current setup code *always* uses a log file
> > of the appropriate level. It's a fallback to catch future changes.
> 
> Well, at least make it "<no log was in use>" (with the brackets) to make the
> message easier to read.

Fine by me.

> >> How about:
> >
> > No. I like Igors approach better.
> > For yours, we end up with duplicate resources : you should split the
> > 'installation incomplete' and the log to check into two separate
> > messages.
> 
> Then you have to tangle with temporary buffers to combine them.

That should be trivial with the string classes, and if it's not, then
that sounds like a call for a method or operator on cistring.

> > Lastly we should be looking at how we pull conditionals out of
> > LogFile,
> > not inserting them.
> 
> I can't see how else to acheive the result I intended without major
> restructuring.

So, take steps toward the overall result, not away from it.

Rob

-- 
GPG key available at: <http://users.bigpond.net.au/robertc/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]