Skip to content

Conversation

multimeric
Copy link

@multimeric multimeric commented Aug 1, 2025

Fixes #284.

Changes

  • Put the environment name into a variable which would allow it to be changed into a CLI flag
  • Use $pkg_manager instead of conda throughout
  • Don't activate the created environment at all, but rather conda run --name commands inside it. This avoids any need for shell configuration, activating the env or needing to locate it. This also makes it more shell independent.
  • Fix spelling mistake
  • Add terminating newline
  • Make the script executable so it can be run as ./install_bindcraft.sh

Testing

I've tested this on our HPC with ./install_bindcraft.sh -p micromamba. This tests the case where the conda installation is in a different location to the environment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug in install script with non-standard environment location
1 participant