Skip to content

Update alpine (3.4 -> 3.5) & java (8u111 -> 8u121) #102

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 1 commit into from
Closed

Update alpine (3.4 -> 3.5) & java (8u111 -> 8u121) #102

wants to merge 1 commit into from

Conversation

viartemev
Copy link
Contributor

No description provided.

@viartemev viartemev changed the title Update java version from 8u111 to 8u112 Update alpine (3.4 -> 3.5) & java (8u111 -> 8u121) Mar 3, 2017
@yosifkit
Copy link
Member

yosifkit commented Mar 3, 2017

We had tried moving to Alpine 3.5 when it came out, but reverted it with #92 since it could be a breaking change for many users. That said, since this is the simplest way to get the Java 8 update via Alpine packages, I think we might be ok to bump the Alpine version since this will also be a bump on the Java version.

While I don't like breaking people but I am guessing the CVE's mentioned in the Oracle patch advisory are more important and the users have a specific version jump to test the updated Alpine version.

cc @tianon

@yosifkit
Copy link
Member

yosifkit commented Mar 3, 2017

If this is accepted, could you also update the update.sh script to ensure the right Alpine version when run by the bot?

[8]='3.4'

@viartemev
Copy link
Contributor Author

viartemev commented Mar 4, 2017

@yosifkit done, I have created new pull request with updated update.sh script.

@viartemev viartemev closed this Mar 4, 2017
@tianon
Copy link
Member

tianon commented Mar 6, 2017

@viartemev in the future, please simply push new changes to the existing branch on your fork -- GitHub will automatically update this PR, which causes less churn in all our inboxes 👍

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.

3 participants