From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <insight-return-5444-listarch-insight=sources.redhat.com@sourceware.org>
Received: (qmail 20496 invoked by alias); 5 Aug 2005 14:01:06 -0000
Mailing-List: contact insight-help@sourceware.org; run by ezmlm
Precedence: bulk
List-Subscribe: <mailto:insight-subscribe@sourceware.org>
List-Archive: <http://sourceware.org/ml/insight/>
List-Post: <mailto:insight@sourceware.org>
List-Help: <mailto:insight-help@sourceware.org>, <http://sourceware.org/ml/#faqs>
Sender: insight-owner@sourceware.org
Received: (qmail 20466 invoked by uid 22791); 5 Aug 2005 14:00:59 -0000
Received: from host217-40-213-68.in-addr.btopenworld.com (HELO SERRANO.CAM.ARTIMI.COM) (217.40.213.68)
    by sourceware.org (qpsmtpd/0.30-dev) with ESMTP; Fri, 05 Aug 2005 14:00:59 +0000
Received: from mace ([192.168.1.25]) by SERRANO.CAM.ARTIMI.COM with Microsoft SMTPSVC(6.0.3790.1830);
	 Fri, 5 Aug 2005 15:01:06 +0100
From: "Dave Korn" <dave.korn@artimi.com>
To: "'Andrew STUBBS'" <andrew.stubbs@st.com>,
	<insight@sources.redhat.com>
Subject: RE: MinGW setjmp SEGV
Date: Fri, 05 Aug 2005 14:01:00 -0000
MIME-Version: 1.0
Content-Type: text/plain;
	charset="us-ascii"
Content-Transfer-Encoding: 7bit
In-Reply-To: <op.su1e3pz3o669wz@terrorhawk.bri.st.com>
Message-ID: <SERRANOWm1izAKlydPA000000bd@SERRANO.CAM.ARTIMI.COM>
X-SW-Source: 2005-q3/txt/msg00053.txt.bz2

----Original Message----
>From: Andrew STUBBS
>Sent: 05 August 2005 14:44

> Hi,
> 
> I have been trying to build Insight 6.1 

> There are a number of patches required to make it build, but most are
> trivial and may not be relevant since 6.1 so I shall not post them.

> However, the most serious failure was caused by broken code in the tcl dll
> screwing up the SEH and causing setjmp to fail. 

> The attached patch (code copied from sourceforge) is sufficient to fix the
> issue. Note that I have included the diff of the whole file and it
> includes one of the minor changes mentioned above.

  That'll be the deleted _INSIDE_CYGWIN_ then?

> 
> Hopefully this patch will help others who have hit the same problem.


    Hi Andrew,

  I was looking through this area just the other day
(http://sourceware.org/ml/insight/2005-q3/msg00021.html), and ISTM that the
substance of this patch is already in CVS HEAD.  The changes appear to have
been added in rev1.9
(http://sources.redhat.com/cgi-bin/cvsweb.cgi/src/tcl/win/tclWin32Dll.c?cvsr
oot=src#rev1.9) when tcl 8.4.1 was merged from upstream.  Just FYI.


    cheers,
      DaveK
-- 
Can't think of a witty .sigline today....