Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

If the executable name is tsvtk the -t option is set #117

Merged
merged 1 commit into from
Jan 4, 2021

Conversation

bsipos
Copy link
Contributor

@bsipos bsipos commented Dec 10, 2020

This is just a simple hack, which sets the -t option if the name of the executable is tsvtk (a symbolic link is practical).
I have found this a more flexible way to set -t as compared to the environmental variable.
Then it would be trivial to tweak the bioconda recipe to create a symbolic link during install with the appropriate name.

@bsipos
Copy link
Contributor Author

bsipos commented Jan 4, 2021

@shenwei356 would you consider this PR for merging?

@shenwei356 shenwei356 merged commit 897d6aa into shenwei356:master Jan 4, 2021
@shenwei356
Copy link
Owner

Sorry, I missed this PR.

It's a nice trick.

@bsipos
Copy link
Contributor Author

bsipos commented Jan 4, 2021

Thanks for the merge! I will tweak the biconda recipe after this makes it to the next release.

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.

2 participants