On 10/19/20 10:03 AM, Tobias Burnus wrote: > Thomas Schwinge and Joseph convinced me that 'rootme' only makes sense > for in-tree testing and, hence, does not need (or: should not) be set in > site.exp. > > Thus, if it is not set, we have to check its existence before using it – > to avoid similar TCL errors. > Hence, I updated the patch to check also for 'rootme'. > > OK? > > Tobias > > On 10/19/20 11:46 AM, Tobias Burnus wrote: >> In a --disable-libsanitizer build, I see errors such as: >> g++.sum:ERROR: can't read "asan_saved_library_path": no such variable >> >> I believe the following patch is the right way to solve this. >> OK? >> >> Tobias >> > ----------------- > Mentor Graphics (Deutschland) GmbH, Arnulfstraße 201, 80634 München / > Germany > Registergericht München HRB 106955, Geschäftsführer: Thomas Heurung, > Alexander Walter > > san-test-v2.diff > > testsuite: Avoid TCL errors when rootme or ASAN/TSAN/UBSAN is not avail > > * g++.dg/guality/guality.exp: > * gcc.dg/guality/guality.exp: > * gfortran.dg/guality/guality.exp: > * lib/asan-dg.exp: > * lib/tsan-dg.exp: > * lib/ubsan-dg.exp: > > gcc/testsuite/g++.dg/guality/guality.exp | 2 +- > gcc/testsuite/gcc.dg/guality/guality.exp | 2 +- > gcc/testsuite/gfortran.dg/guality/guality.exp | 2 +- > gcc/testsuite/lib/asan-dg.exp | 6 ++++-- > gcc/testsuite/lib/tsan-dg.exp | 6 ++++-- > gcc/testsuite/lib/ubsan-dg.exp | 6 ++++-- > 6 files changed, 15 insertions(+), 9 deletions(-)
OK with ChangeLog entry completed. jeff