Skip to content

Commit

Permalink
Remove (DY)LD_LIBRARY_PATH for TestCommonInstall
Browse files Browse the repository at this point in the history
(DY)LD_LIBRARY_PATH is not necessary anymore to use pydrake since RPath is set
on Linux and libraries know their dependencies location on MacOS.
  • Loading branch information
Francois Budin committed Oct 23, 2017
1 parent 7d20049 commit 37aadef
Showing 1 changed file with 1 addition and 9 deletions.
10 changes: 1 addition & 9 deletions drake/bindings/python/pydrake/test/testCommonInstall.py
Original file line number Diff line number Diff line change
Expand Up @@ -34,20 +34,12 @@ def testDrakeFindResourceOrThrowInInstall(self):
os.remove(element)
self.assertEqual(os.listdir("."), [tmp_folder])

# Set the correct PYTHONPATH and (DY)LD_LIBRARY_PATH to use the
# correct pydrake module.
# Set the correct PYTHONPATH to use the correct pydrake module.
env_python_path = "PYTHONPATH"
if sys.platform == 'darwin':
env_ld_library = "DYLD_LIBRARY_PATH"
else:
env_ld_library = "LD_LIBRARY_PATH"
tool_env = dict(os.environ)
tool_env[env_python_path] = os.path.abspath(
os.path.join(tmp_folder, "lib", "python2.7", "site-packages")
)
tool_env[env_ld_library] = os.path.abspath(
os.path.join(tmp_folder, "lib")
)
data_folder = os.path.join(tmp_folder, "share", "drake", "drake")
output_path = subprocess.check_output(
["python",
Expand Down

0 comments on commit 37aadef

Please sign in to comment.