search

facebookarchive / facebook-clang-plugins

Plugins to clang-analyzer and clang-frontend
MIT License
482 stars 85 forks source link

Add support for correctly reading the 'TMPDIR' environment variable #26

Closed aytey closed 3 years ago

aytey commented 3 years ago

Infer's install-sh knows to "correctly" read from the TMPDIR environment variable:

However, this was not supported by clang/setup.sh.

I haven't overlooked the existence of CLANG_TMP_DIR, but I had (initially) set TMPDIR hoping Infer's build system would honour it (it didn't) -- consequently, I thought it would be useful for future users if facebook-clang-plugins honoured this.

Signed-off-by: Andrew V. Jones andrewvaughanj@gmail.com