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

added java chef recipe... #337

Closed
wants to merge 1 commit into from
Closed

added java chef recipe... #337

wants to merge 1 commit into from

Conversation

ZogStriP
Copy link
Member

@ZogStriP ZogStriP commented Mar 3, 2013

... and told vagrant to use it.

This will automatically install JRE 6 when starting the vagrant box.

This will make JsHint work with Guard.

and told vagrant to use it
@discoursebot
Copy link

You've signed the CLA, ZogStriP. Thank you! This pull request is ready for review.

@ZogStriP
Copy link
Member Author

ZogStriP commented Mar 3, 2013

Alternatively, another solution would be to update the vagrant box provided by Discourse.

@eviltrout
Copy link
Contributor

Just tried this and it seems to work - I wonder if we need all the windows stuff though, since we assume the VM is linux right now?

@ZogStriP
Copy link
Member Author

ZogStriP commented Mar 4, 2013

From what I understand, the java recipe requires the windows recipe which requires the chef_handler recipe. I'm not sure how to remove that dependency.

Anyway, I believe the best way would be to update the vagrant box by including java. I'll try to give it a shot.

@eviltrout
Copy link
Contributor

We're planning on releasing a new VM late this week after we change the
seed data, so I have it on my list anyway :)

On Mon, Mar 4, 2013 at 6:15 PM, Régis Hanol notifications@github.comwrote:

From what I understand, the java recipe requires the windows recipe
which requires the chef_handler recipe. I'm not sure how to remove that
dependency.

Anyway, I believe the best way would be to update the vagrant box by
including java. I'll try to git it a shot.


Reply to this email directly or view it on GitHubhttps://github.com//pull/337#issuecomment-14413061
.

@ZogStriP
Copy link
Member Author

ZogStriP commented Mar 4, 2013

Excellent ! :)

@eviltrout
Copy link
Contributor

Closing this for now due to better VM coming later this week :)

@eviltrout eviltrout closed this Mar 5, 2013
@ZogStriP ZogStriP deleted the jre-chef-recipe branch March 22, 2013 09:49
CvX pushed a commit that referenced this pull request May 19, 2021
FEATURE: support oneboxing of pdf files
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants