Skip to content

new role variables to allow setting custom scala+sbt install folders #2

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

Closed
wants to merge 3 commits into from

Conversation

lhoss
Copy link

@lhoss lhoss commented Mar 2, 2016

No description provided.

@@ -11,5 +11,5 @@ galaxy_info:
- trusty
categories:
- development
dependencies:
- { role: "azavea.java" }
#dependencies:
Copy link
Contributor

Choose a reason for hiding this comment

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

Are you installing java6-runtime-headles or equivalent on your own? The role fails to provision without it.

Copy link
Author

Choose a reason for hiding this comment

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

Indeed, we don't want to hardcode such a base dependency (like java) that is used by almost any framework/tool.

ps: I'm still waiting for any fix to:

Unfort. ansible doesn't yet offer a feature to easily disable 'transient dependencies'.
I created an ansible feature request: ansible/ansible#13215 , but will probably take time to get it builtin (hopefully for v2.1)

Copy link
Author

Choose a reason for hiding this comment

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

@hectcastro would you merge the PR, if undo this disabled java dependency?
ps: Unfort. my issue (ansible/ansible#13215) didn't get enough traction .. at least there's more recent similar requests that would also help, like ansible/ansible#34566

Copy link
Author

Choose a reason for hiding this comment

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

I've now done as I proposed -^ in a new PR, AND also properly documenting the 3 new role vars
-> #4 👍

@lhoss
Copy link
Author

lhoss commented Feb 27, 2018

replaced by #4

@lhoss lhoss closed this Feb 27, 2018
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

Successfully merging this pull request may close these issues.

2 participants