search

steleman / address-sanitizer

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

Remove the (A|M|T|L)SAN_SYMBOLIZER_PATH env variables #341

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
I think this ugliness needs to go. We've had the "external_symbolizer_path" 
flag for a while. We should either keep ONE env variable that would affect all 
tools (LLVM_SYMBOLIZER_PATH), or leave the flag as the only option.

I propose to make sanitizers print a deprecation warning when one of those 
variables is set, then stop supporting them starting with the next Clang 
release.

Original issue reported on code.google.com by earth...@google.com on 17 Sep 2014 at 5:59

GoogleCodeExporter commented 9 years ago

Original comment by earth...@google.com on 17 Sep 2014 at 6:00

GoogleCodeExporter commented 9 years ago 
gcc crowd won't like LLVM_SYMBOLIZER_PATH name

Original comment by dvyu...@google.com on 17 Sep 2014 at 6:05

GoogleCodeExporter commented 9 years ago 
SANITIZER_SYMBOLIZER_PATH also works

Original comment by earth...@google.com on 17 Sep 2014 at 6:06

GoogleCodeExporter commented 9 years ago 
I agree with your proposal. Let's switch to SANITIZER_SYMBOLIZER_PATH (I think 
that special env variable might be handy sometimes).

Original comment by samso...@google.com on 17 Sep 2014 at 6:10

GoogleCodeExporter commented 9 years ago 
Can I throw in a shorter version? SAN_SYMBOLIZER_PATH

env space is not for free, and this just strips the first char, so it's easier 
to fix in scripts and remember, I think.

Original comment by reini.urban on 17 Sep 2014 at 8:27