public inbox for cygwin@cygwin.com
 help / color / mirror / Atom feed
* [PATCH] Prevent g++ testsuite crahing on cygwin
@ 2001-03-15 15:47 Billinghurst, David (CRTS)
  0 siblings, 0 replies; only message in thread
From: Billinghurst, David (CRTS) @ 2001-03-15 15:47 UTC (permalink / raw)
  To: 'gcc-patches@gcc.gnu.org'; +Cc: 'cygwin@cygwin.com'

The g++ testsuite fails on cygwin.  The root cause is a bug in
old-dejagnu.exp, where it attempts to delete any output generated when the
test case is executed, in addition to deleting the executable.  See
http://gcc.gnu.org/ml/gcc-bugs/2001-03/msg00453.html .

With this patch, the g++ testsuite works on NT4/cygwin.  Bootstrapped 3.0
branch on cygwin and irix6.5

2001-03-15  David Billinghurst  (David.Billinghurst@riotinto.com)

	* lib/old-dejagnu.exp:  Don't try and delete output from executable

Index: old-dejagnu.exp
===================================================================
RCS file: /cvs/gcc/egcs/gcc/testsuite/lib/old-dejagnu.exp,v
retrieving revision 1.22
diff -u -r1.22 old-dejagnu.exp
--- old-dejagnu.exp     2001/02/08 02:29:38     1.22
+++ old-dejagnu.exp     2001/03/15 23:13:15
@@ -574,9 +574,9 @@
            }
            $status "$name $pattern Execution test"
        }
+    } else {
+        verbose "deleting $output"
+        remote_file build delete $output
     }
-
-    verbose "deleting $output"
-    remote_file build delete $output
     return 0
 }

--
Want to unsubscribe from this list?
Check out: http://cygwin.com/ml/#unsubscribe-simple

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2001-03-15 15:47 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2001-03-15 15:47 [PATCH] Prevent g++ testsuite crahing on cygwin Billinghurst, David (CRTS)

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).