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

Documentation for dependency resolvers #1296

Merged
merged 6 commits into from Dec 11, 2015
Merged

Documentation for dependency resolvers #1296

merged 6 commits into from Dec 11, 2015

Conversation

pvanheus
Copy link
Contributor

Documentation largely derived from my blog post (http://pvh.wp.sanbi.ac.za/2015/10/09/how-galaxy-resolves-dependencies-or-not/) on how dependency resolvers are and can be configured for Galaxy.

@@ -1,7 +1,10 @@
<dependency_resolvers>
<!-- the default configuration, first look for dependencies installed from the toolshed -->
<tool_shed_packages />
<!-- then look for env.sh files profile according to the "galaxy packages" schema -->
<!-- then look for env.sh files in directories according to the "galaxy packages" schema.
These resolvers can take a base_path attribute to specify where to look for
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can't find the code where base_path attribute should be parsed, lib/galaxy/tools/deps/__init__.py seems to always use config.tool_dependency_dir .

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Its used in lib/galaxy/tools/deps/resolvers/resolver_mixins.py in the UsesToolDependencyDirMixin which is used by the galaxy_packages resolver.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@nsoranzo
Copy link
Member

Thanks @pvanheus, nice work!

* Use modulepath in both environment modules resolvers
* Export MODULEPATH in generated shell command
* Add logging for operation of dependency resolvers
    * Replace {} and .format with %s and % to preserve Python 2.6 compatibility
    * Add a modulepath= paranter to the fake resolver (Bunch object) used to test the modules dependency resolver
@jmchilton
Copy link
Member

Utterly and wonderfully fantastic @pvanheus!

jmchilton added a commit that referenced this pull request Dec 11, 2015
Documentation for dependency resolvers
@jmchilton jmchilton merged commit 6580d51 into galaxyproject:dev Dec 11, 2015
@jmchilton jmchilton modified the milestone: 16.01 Jan 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants