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

Improve handling of a missing R environment. #2215

Merged
merged 1 commit into from Apr 21, 2016

Conversation

Projects
None yet
5 participants
@davebx
Copy link
Contributor

commented Apr 21, 2016

Previously, the error message 'Missing R environment. Please check your specified R installation exists.' would be displayed in an obscure location in the paster log. With this change, galaxy will set the tool dependency to Error and display the error message in the admin panel.

The lack of a displayed error led to @nturaga and I spending an extra 30 minutes tracking down a misconfigured dependency on Tuesday.

Improve handling of a missing R environment.
Previously, the error message 'Missing R environment. Please check
your specified R installation exists.' would be displayed in an
obscure location in the log. With this change, galaxy will set
the tool dependency to Error and display the error message in the
admin panel.

@galaxybot galaxybot added the triage label Apr 21, 2016

@galaxybot galaxybot added this to the 16.07 milestone Apr 21, 2016

@jmchilton jmchilton merged commit d0028bd into galaxyproject:dev Apr 21, 2016

4 checks passed

api test Build finished.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
framework test Build finished.
Details
toolshed test Build finished.
Details
@jmchilton

This comment has been minimized.

Copy link
Member

commented Apr 21, 2016

Thanks for the fix @davebx

@nturaga

This comment has been minimized.

Copy link
Contributor

commented Apr 21, 2016

+1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.