Skip to content

Install Chimera in a subdirectory to avoid its dependencies being added to the environment#1413

Merged
boegel merged 1 commit intoeasybuilders:developfrom
sebth:chimera_subdirectory
May 21, 2018
Merged

Install Chimera in a subdirectory to avoid its dependencies being added to the environment#1413
boegel merged 1 commit intoeasybuilders:developfrom
sebth:chimera_subdirectory

Conversation

@sebth
Copy link
Copy Markdown
Contributor

@sebth sebth commented Apr 30, 2018

I noticed that when loading Chimera after loading Python, Python starts picking up the Tcl/Tk version that comes bundled with Chimera instead of the correct one. This PR fixes that by installing Chimera in a subdirectory so that the bundled dependencies are not added to PATH, LD_LIBRARY_PATH, etc. A symlink is added from 'bin/chimera' in the root directory to 'bin/chimera' in subdirectory, so that a 'chimera' command will exist in PATH. Chimera will still find its bundled dependencies, because it uses a wrapper startup script that adds them to the environment.

This method is similar to what they describe in the installation instructions at http://www.cgl.ucsf.edu/chimera/data/downloads/1.11.2/linux.html.

Installing Chimera in a subdirectory avoids its bundled dependencies
being added to PATH, CPATH, LD_LIBRARY_PATH, etc. when Chimera's module
is loaded.  Chimera already comes with a startup wrapper script that
handles adding its bundled dependencies to the environment.
@sebth sebth changed the title Install Chimera in a subdirectory to avoid its dependencies being added to Install Chimera in a subdirectory to avoid its dependencies being added to the environment Apr 30, 2018
@boegel boegel added this to the 3.6.1 milestone May 21, 2018
@boegel boegel added the bug fix label May 21, 2018
Copy link
Copy Markdown
Member

@boegel boegel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@boegel
Copy link
Copy Markdown
Member

boegel commented May 21, 2018

Thanks a lot for your contribution @sebth, sorry about the long wait for feedback...

This looks perfect & well documented, I tested this myself and it works as expected, so good to go.

@boegel boegel merged commit 6afab5d into easybuilders:develop May 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants