Mailing-List: contact cygwin-apps-help AT cygwin DOT com; run by ezmlm
Sender: cygwin-apps-owner AT cygwin DOT com
List-Subscribe: <mailto:cygwin-apps-subscribe AT cygwin DOT com>
List-Archive: <http://sources.redhat.com/ml/cygwin-apps/>
List-Post: <mailto:cygwin-apps AT cygwin DOT com>
List-Help: <mailto:cygwin-apps-help AT cygwin DOT com>, <http://sources.redhat.com/lists.html#faqs>
Mail-Followup-To: cygwin-apps AT cygwin DOT com
Delivered-To: mailing list cygwin-apps AT cygwin DOT com
Date: Wed, 15 May 2002 08:26:36 -0400
From: Christopher Faylor <cgf AT redhat DOT com>
To: cygwin-apps AT cygwin DOT com
Subject: Re: Setup 2.218.2.6 [Was: Re: Setup 2.218.2.4] (new version uploaded)
Message-ID: <20020515122636.GB11699@redhat.com>
Reply-To: cygwin-apps AT cygwin DOT com
Mail-Followup-To: cygwin-apps AT cygwin DOT com
References: <FC169E059D1A0442A04C40F86D9BA7600C60BA AT itdomain003 DOT itdomain DOT net DOT au>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <FC169E059D1A0442A04C40F86D9BA7600C60BA@itdomain003.itdomain.net.au>
User-Agent: Mutt/1.3.23.1i

On Wed, May 15, 2002 at 05:07:53PM +1000, Robert Collins wrote:
>
>
>> -----Original Message-----
>> From: Christopher Faylor [mailto:cgf AT redhat DOT com] 
>> Sent: Wednesday, May 15, 2002 8:55 AM
>
>
>> The attached patch seems to fix this and some other problems.
>> 
>> I've uploaded a new version to sourceware.
>> 
>> At least some of the below should be useful on the trunk, too.
>
>Thanks heaps Chris. I appreciate this. (BTW: how did you reproduce the
>fault?)

Having a setup.ini at the "root" of the locally installed directory seemed
to cause one problem.  Then, once setup had an error then the same error was
reported for every setup.ini parsed.

Including '\n' in the error condition in the parser also seemed to make
the parser get out of sync, causing the cascading errors that people
were reporting.

>I notice that you've increased the stack size for the parser - this
>shouldn't have been needed with the right hand recursion removed. Did
>you observe continued overflows or was this preventative?

My erroneous setup.ini had a requires line which depended on *a lot* of
packages and it caused yacc to have a stack overflow.  It took me a couple
of hours to realize that the stack overflow was entirely due to my stupid
setup.ini and not to any code changes that I was making.

It seemed like a win to just increase the stack size.  I don't know why
it isn't getting increased automatically, though.  From inspecting the
code, it seems like it should have been.

cgf