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

bedtools mapBed.xml tool is invalid #148

Closed
nsoranzo opened this issue May 21, 2015 · 6 comments · Fixed by #150
Closed

bedtools mapBed.xml tool is invalid #148

nsoranzo opened this issue May 21, 2015 · 6 comments · Fixed by #150
Assignees
Labels

Comments

@nsoranzo
Copy link
Member

After latest push to the MTS, https://toolshed.g2.bx.psu.edu/view/iuc/bedtools/457b09031d57 reports that mapBed.xml is invalid:

This file requires an entry for "'input'" in the tool_data_table_conf.xml file. Upload a file named tool_data_table_conf.xml.sample to the repository that includes the required entry to correct this error.

which does not make much sense for me.

@nsoranzo
Copy link
Member Author

Unfortunately PR #150 fixed mapBed.xml , but broke mergeBed.xml . From https://testtoolshed.g2.bx.psu.edu/repository?repository_id=7f940896abcf000c&changeset_revision=bdb4509d9482 :

This file requires an entry for "'inputA'" in the tool_data_table_conf.xml file. Upload a file named tool_data_table_conf.xml.sample to the repository that includes the required entry to correct this error.

@bgruening
Copy link
Member

True, thanks for reporting. It seems that neither planemo l and planemo t catches these kind of errors.

bgruening added a commit that referenced this issue May 23, 2015
@bgruening
Copy link
Member

@nsoranzo next try #148
I have removed the macro and using yield now to manually adopt for the correct input value.

@nsoranzo
Copy link
Member Author

@bgruening The commit LGTM, can you open a PR?

@bgruening
Copy link
Member

@nsoranzo #161 Thanks for reviewing!

hexylena added a commit that referenced this issue May 25, 2015
@nsoranzo
Copy link
Member Author

Fixed by #161 and uploaded to the MTS.

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

Successfully merging a pull request may close this issue.

2 participants