search

gopalshankar / address-sanitizer

Automatically exported from code.google.com/p/address-sanitizer
0 stars 0 forks source link

Non-portable test #285

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
ignore_lib0.cc testcase is not portable. This test overrides LD_LIBRARY_PATH:
  RUN: LD_LIBRARY_PATH=%T not %t 2>&1 ...
On my system this fails with an error
  /usr/lib/x86_64-linux-gnu/libstdc++.so.6: version `GLIBCXX_3.4.20' not found 
because correct version of libstdc++ is provided via LD_LIBRARY_PATH.

Original issue reported on code.google.com by tetra2...@gmail.com on 31 Mar 2014 at 12:18

GoogleCodeExporter commented 9 years ago 
Will this work if we re-write this as LD_LIBRARY_PATH=%T:$LD_LIBRARY_PATH?
There is a separate bug tracker for TSan btw 
(https://code.google.com/p/thread-sanitizer/issues/list)

Original comment by samso...@google.com on 31 Mar 2014 at 2:26

GoogleCodeExporter commented 9 years ago 
I thought about this but if LD_LIBRARY_PATH was empty, you'll inadverently add 
curdir into list of library paths which is probably undesirable.

Original comment by tetra2...@gmail.com on 31 Mar 2014 at 2:31

GoogleCodeExporter commented 9 years ago 
The canonical way for POSIX shells would probably be 
LD_LIBRARY_PATH=%T${LD_LIBRARY_PATH:+:$LD_LIBRARY_PATH}

Original comment by tetra2...@gmail.com on 31 Mar 2014 at 2:34

GoogleCodeExporter commented 9 years ago 
Submitted r205300.

Original comment by samso...@google.com on 1 Apr 2014 at 11:58