Skip to content
This repository has been archived by the owner on Oct 2, 2020. It is now read-only.

Fix #88 in orig repo about creating recursive symlink for .fai file #91

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

samuell
Copy link
Contributor

@samuell samuell commented May 23, 2016

This fixes #88

@mr-c
Copy link
Member

mr-c commented May 23, 2016

LGTM, though why hasn't Travis chimed in?

@portah any thoughts on the fix?

@portah
Copy link
Collaborator

portah commented Jun 2, 2016

I'm not quite sure that it was a bug in CWL, the .fai was commented out I think because that samtools version worked differently. Is that fix for the same samtools docker version?

About travis - travis is just syntax but not an actual run, it does not create any outputs

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

Successfully merging this pull request may close these issues.

samtools faidx replaces fasta file with symlink pointing to itself
3 participants