public inbox for gcc@gcc.gnu.org
 help / color / mirror / Atom feed
* A patch for dejagnu
@ 1998-03-11  9:39 H.J. Lu
  1998-03-12 17:00 ` Jim Wilson
  0 siblings, 1 reply; 4+ messages in thread
From: H.J. Lu @ 1998-03-11  9:39 UTC (permalink / raw)
  To: egcs

Hi,

Some people have reported problems with the latest dejagnu and
egcs that dejagnu cannot find the correct libstdc++ and libio.
Here is a patch which seems to work for me.


-- 
H.J. Lu (hjl@gnu.org)
---
--- libgloss.exp.orig	Wed Mar 11 08:26:07 1998
+++ libgloss.exp	Wed Mar 11 09:33:34 1998
@@ -425,14 +425,17 @@
 
 
     # search for the top level multilib directory
-    set multitop [lookfor_file "${comp_base_dir}" "${target_alias}"]
+    set multitop [lookfor_file "${comp_base_dir}" "libraries"]
     if { $multitop == "" } {
-	set multitop [lookfor_file "${comp_base_dir}" "libraries"]
-	if { $multitop == "" } {
-	    set multitop "[lookfor_file ${comp_base_dir} gcc/xgcc]"
-	    if { $multitop != "" } {
-		set multitop [file dirname [file dirname $multitop]];
-	    } else {
+	set multitop "[lookfor_file ${comp_base_dir} gcc/xgcc]"
+	if { $multitop != "" } {
+	    set multitop [file dirname [file dirname $multitop]];
+	} else {
+	    # We search ${target_alias} last since it is only valid
+	    # when we are running dejagnu on an installed gcc. We
+	    # don't want to get it by accident.
+	    set multitop [lookfor_file "${comp_base_dir}" "${target_alias}"]
+	    if { $multitop == "" } {
 		return ""
 	    }
 	}

^ permalink raw reply	[flat|nested] 4+ messages in thread
* Re: none
@ 1998-05-28  6:19 Jeffrey A Law
  1998-05-28 17:47 ` A patch for dejagnu H.J. Lu
  0 siblings, 1 reply; 4+ messages in thread
From: Jeffrey A Law @ 1998-05-28  6:19 UTC (permalink / raw)
  To: Joern Rennecke; +Cc: oliva, egcs, wilson

  In message < 199805272005.VAA20442@phal.cygnus.co.uk >you write:
  > >   > So I came up with a solution that would keep gcc using TMPDIR for its
  > >   > temporaries, for speed, and would allow me to specify different
  > >   > directories for each run of dejagnu.  Perhaps the patch should use
  > >   > TMPDIR if TESTSUITE_TMPDIR is not set:
  > > Ah.  In that case, maybe the right way to go is to put temporary
  > > files in the testsuite object directory itself since you'd have
  > > different testsuite object directories for the different configs
  > > right?
  > 
  > No, this doesn't work when you want to run different tests from the
  > same (e.g. multilibbed) directory.
That's only an issue for folks doing parallel "make check" for multilibbed
configs.  Granted it'd be nice to handle that, but I don't think that's
the most important issue at hand, and it certainly doesn't work right
now!

If we want to handle that case, then I might suggest we have the
tmpdir be a subdir of the testsuite object directory.  The name of
the subdir would be derived from the multilib options.

So, does someone want to write a patch for that?  Or do we go with
a partial solution?

jeff


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~1998-05-28 17:47 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
1998-03-11  9:39 A patch for dejagnu H.J. Lu
1998-03-12 17:00 ` Jim Wilson
1998-03-16 20:04   ` Jeffrey A Law
1998-05-28  6:19 none Jeffrey A Law
1998-05-28 17:47 ` A patch for dejagnu H.J. Lu

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).