Skip to content

fix tophat fusion post#507

Merged
davebx merged 2 commits intomasterfrom
bug
Jun 13, 2018
Merged

fix tophat fusion post#507
davebx merged 2 commits intomasterfrom
bug

Conversation

@bgruening
Copy link
Member

I wonder if it ever worked.

@davebx
Copy link
Contributor

davebx commented Jun 12, 2018

@bgruening it would have worked the other way, but this brings it up to better standards. In fact the ./ in front of blast is not technically necessary, since ln with no path for the target will put it in the CWD.

@bgruening
Copy link
Member Author

@davebx thanks Dave. I think the problem was a space in the blast db name or something like that.

@davebx
Copy link
Contributor

davebx commented Jun 13, 2018

@bgruening that makes perfect sense, and illustrates why we decided to make quoted paths a standard. Thanks for the fix.

@davebx davebx merged commit c016491 into master Jun 13, 2018
@martenson martenson changed the title this tool seems to be broken fix tophat fusion post Jun 13, 2018
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