hubert.reinterpretcast created this revision.
hubert.reinterpretcast added reviewers: xingxue, jasonliu, sfertile.
Herald added a reviewer: serge-sans-paille.
Herald added subscribers: cfe-commits, jsji.
Herald added a project: clang.

When building the `check-all` target on AIX, lit produces

  warning: unable to inject shared library path on 'AIX'

This patch addresses this. `LIBPATH` is the environment variable of interest on 
AIX. Newer versions of AIX may consider `LD_LIBRARY_PATH`, but only when 
`LIBPATH` is unset.


Repository:
  rC Clang

https://reviews.llvm.org/D59741

Files:
  test/Unit/lit.cfg.py


Index: test/Unit/lit.cfg.py
===================================================================
--- test/Unit/lit.cfg.py
+++ test/Unit/lit.cfg.py
@@ -42,6 +42,8 @@
         yield 'DYLD_LIBRARY_PATH'
     elif platform.system() == 'Windows':
         yield 'PATH'
+    elif platform.system() == 'AIX':
+        yield 'LIBPATH'
 
 for shlibpath_var in find_shlibpath_var():
     # in stand-alone builds, shlibdir is clang's build tree


Index: test/Unit/lit.cfg.py
===================================================================
--- test/Unit/lit.cfg.py
+++ test/Unit/lit.cfg.py
@@ -42,6 +42,8 @@
         yield 'DYLD_LIBRARY_PATH'
     elif platform.system() == 'Windows':
         yield 'PATH'
+    elif platform.system() == 'AIX':
+        yield 'LIBPATH'
 
 for shlibpath_var in find_shlibpath_var():
     # in stand-alone builds, shlibdir is clang's build tree
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to