From d11994a8c39186392711d06554dc5baab5dc3b72 Mon Sep 17 00:00:00 2001 From: Steven Bosscher Date: Thu, 18 Dec 2003 19:39:57 +0000 Subject: [PATCH] re PR other/12009 (g++.dg, compat.exp fail to crosscompile testcases) PR other/12009 * g++.dg/compat/compat.exp: Do not set LD_LIBRARY_PATH when testing a cross compiler, it causes spurious compile failures. * lib/g++.exp: Likewise. Co-Authored-By: Dan Kegel From-SVN: r74789 --- gcc/testsuite/ChangeLog | 8 ++++++++ gcc/testsuite/g++.dg/compat/compat.exp | 12 +++++++----- gcc/testsuite/lib/g++.exp | 13 ++++++++----- 3 files changed, 23 insertions(+), 10 deletions(-) diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 009c8e9c691..2adba02f1a9 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,11 @@ +2003-12-18 Steven Bosscher + Dan Kegel + + PR other/12009 + * g++.dg/compat/compat.exp: Do not set LD_LIBRARY_PATH when + testing a cross compiler, it causes spurious compile failures. + * lib/g++.exp: Likewise. + 2003-12-18 Kriang Lerdsuwanakij PR c++/13262 diff --git a/gcc/testsuite/g++.dg/compat/compat.exp b/gcc/testsuite/g++.dg/compat/compat.exp index 1adb4569c6d..94da38c77de 100644 --- a/gcc/testsuite/g++.dg/compat/compat.exp +++ b/gcc/testsuite/g++.dg/compat/compat.exp @@ -43,11 +43,13 @@ proc compat-fix-library-path { } { global ld_library_path # See comments in lib/g++.exp for why this is needed. - setenv LD_LIBRARY_PATH $ld_library_path - setenv SHLIB_PATH $ld_library_path - setenv LD_LIBRARYN32_PATH $ld_library_path - setenv LD_LIBRARY64_PATH $ld_library_path - setenv DYLD_LIBRARY_PATH $ld_library_path + if {![is_remote target]} { + # See comments in lib/g++.exp for why this is needed. + setenv LD_LIBRARY_PATH $ld_library_path + setenv SHLIB_PATH $ld_library_path + setenv LD_LIBRARYN32_PATH $ld_library_path + setenv LD_LIBRARY64_PATH $ld_library_path + } } # diff --git a/gcc/testsuite/lib/g++.exp b/gcc/testsuite/lib/g++.exp index 1fcc38f9fa2..8b30b60af14 100644 --- a/gcc/testsuite/lib/g++.exp +++ b/gcc/testsuite/lib/g++.exp @@ -172,11 +172,14 @@ proc g++_link_flags { paths } { # unix.exp -- but that's not an option since it's part of DejaGNU # proper, so we do it here. We really only need to do # this on IRIX, but it shouldn't hurt to do it anywhere else. - setenv LD_LIBRARY_PATH $ld_library_path - setenv SHLIB_PATH $ld_library_path - setenv LD_LIBRARYN32_PATH $ld_library_path - setenv LD_LIBRARY64_PATH $ld_library_path - setenv DYLD_LIBRARY_PATH $ld_library_path + # Doing this does cause trouble when testing cross-compilers. + if {![is_remote target]} { + setenv LD_LIBRARY_PATH $ld_library_path + setenv SHLIB_PATH $ld_library_path + setenv LD_LIBRARYN32_PATH $ld_library_path + setenv LD_LIBRARY64_PATH $ld_library_path + setenv DYLD_LIBRARY_PATH $ld_library_path + } return "$flags" } -- 2.30.2