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.

Sign up for GitHub

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

Jump to bottom

Update OpenJDK version #2178

Merged
merged 1 commit into from
Aug 23, 2017
Merged

Update OpenJDK version #2178

merged 1 commit into from
Aug 23, 2017

Conversation

jtarricone
Copy link
Contributor

Overview

This PR updates the OpenJDK version so that the environment can be built properly.

Connects #2172

Testing Instructions

  • the most thorough test is probably to build a fresh environment (destroying any existing ones if necessary), then go through all the regular steps following that, such as DB setup and the like
  • with a stable environment, scripts/test.sh to run all tests
  • open the app and do some basic action sequences that would invoke the geoprocessor (the TR-55 model would probably suffice, but MapShed could be used instead or in addition to be safest)

hectcastro
hectcastro approved these changes Aug 23, 2017
View reviewed changes
Copy link
Contributor

@hectcastro hectcastro left a comment

Choose a reason for hiding this comment

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

Provisions cleanly from scratch.

@jtarricone jtarricone assigned jtarricone and unassigned hectcastro Aug 23, 2017
@jtarricone jtarricone merged commit 9b11c0d into develop Aug 23, 2017
@rajadain rajadain deleted the fix/jmt/open-jdk-version branch October 6, 2017 21:11
@rajadain rajadain mentioned this pull request Oct 16, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hectcastro hectcastro hectcastro approved these changes

Assignees

@jtarricone jtarricone

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@jtarricone @hectcastro