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

move "convert windows newlines" tool into an other place #9

Open
bgruening opened this issue Jan 24, 2015 · 1 comment
Open

move "convert windows newlines" tool into an other place #9

bgruening opened this issue Jan 24, 2015 · 1 comment

Comments

@bgruening
Copy link
Member

This tool looks strange to me. If the purpose of this tool is to convert windows newlines only for ProteinPilot, as the name suggest, we should include the tr magic into the wrapper of ProteinPilot.

If such a tool is really missing in general we should put it into a text-manipluation repository, e.g. here:
https://github.com/bgruening/galaxytools/tree/master/text_processing/text_processing

Anyone know if this is still a problem or does Galaxy take care of this?
@jmchilton @jj-umn ?

@jmchilton
Copy link
Contributor

Ugh... I don't remember why it is a separate tool. I imagine group2xml used to produce tabular files outputs them with windows newlines - Galaxy will fix uploads but not tabular files created by other tools - but I a not sure why I didn't "fix" them in the tool. Hmm... the folks at MSI may be able to look at the workflows and determine if this needs to be a tool or not still.

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

No branches or pull requests

2 participants