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 25, 2017
1 parent 3f191e0 commit e86f56e
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")
# Call python2 to force using python brew install. Calling python
# system would result in a crash since pydrake was built against
Expand Down

0 comments on commit e86f56e

Please sign in to comment.