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

Force install nss with openjdk in alpine #294

Merged
merged 1 commit into from
Mar 27, 2019
Merged

Force install nss with openjdk in alpine #294

merged 1 commit into from
Mar 27, 2019

Conversation

luiszimmermann
Copy link
Contributor

@luiszimmermann luiszimmermann commented Mar 26, 2019 •
edited by tianon
Loading

Adding nss to the alpine image

Closes #289

@matsewe
Copy link

matsewe commented Mar 27, 2019

The problem concerning the jessie-update repository which led to build failures for VERSION=7 has been fixed; builds should pass now.

@luiszimmermann
Copy link
Contributor Author

Triggering Travis

@matsewe
Copy link

matsewe commented Mar 27, 2019

I guess, another run of update.sh should fix this problem. Most recent version of opendjk-7 for jessie is, according to update.sh, 7u211-2.6.17-1~deb8u1. This is not yet incorporated into the Dockerfile.

@yosifkit yosifkit mentioned this pull request Mar 27, 2019
Closed
@luiszimmermann @tianon
Force install nss in Alpine
79a6e42 
This is a temporary patch while https://bugs.alpinelinux.org/issues/10126 gets resolved properly upstream.
@tianon
Copy link
Member

tianon commented Mar 27, 2019

I've updated this with the minimal change and cancelled every job on Travis (and AppVeyor) that isn't related to the actual fix, so once https://travis-ci.org/docker-library/openjdk/jobs/512206934 is green we're good to go (assuming @yosifkit approves of my approach).

@tianon
Copy link
Member

tianon commented Mar 27, 2019

(Then @docker-library-bot will take care of testing and pushing the rest of the updates as appropriate.)

@tianon
Copy link
Member

tianon commented Mar 27, 2019

Welp, since Travis is having a sad, I've built both of these successfully locally and run the java-ca-certificates test directly and they both now pass.

@tianon tianon merged commit d4ecf99 into docker-library:master Mar 27, 2019
docker-library-bot added a commit to docker-library-bot/official-images that referenced this pull request Mar 27, 2019
@docker-library-bot
Update openjdk
af38b3a 
Changes:

- docker-library/openjdk@311e8bb: Update to 13-ea+12
- docker-library/openjdk@3ba705f: Update to 13-ea+13
- docker-library/openjdk@d42099b: Update to 7u211, debian 7u211-2.6.17-1~deb8u1
- docker-library/openjdk@aea0404: Update to 12
- docker-library/openjdk@b8ce9ef: Update to 8u212, debian 8u212-b01-1~deb9u1
- docker-library/openjdk@d4ecf99: Merge pull request docker-library/openjdk#294 from luiszimmermann/nss-fix
- docker-library/openjdk@79a6e42: Force install nss in Alpine
- docker-library/openjdk@cb138b5: Update to 13-ea+12
@tianon tianon mentioned this pull request Mar 27, 2019
Merged
docker-library-bot added a commit to docker-library-bot/official-images that referenced this pull request Mar 27, 2019
@docker-library-bot
Update openjdk
9056323 
Changes:

- docker-library/openjdk@b7ab504: Merge pull request docker-library/openjdk#296 from infosiftr/12-ga
- docker-library/openjdk@58b0eb3: Update "latest" to OpenJDK 12
- docker-library/openjdk@311e8bb: Update to 13-ea+12
- docker-library/openjdk@3ba705f: Update to 13-ea+13
- docker-library/openjdk@d42099b: Update to 7u211, debian 7u211-2.6.17-1~deb8u1
- docker-library/openjdk@aea0404: Update to 12
- docker-library/openjdk@b8ce9ef: Update to 8u212, debian 8u212-b01-1~deb9u1
- docker-library/openjdk@d4ecf99: Merge pull request docker-library/openjdk#294 from luiszimmermann/nss-fix
- docker-library/openjdk@79a6e42: Force install nss in Alpine
- docker-library/openjdk@cb138b5: Update to 13-ea+12
@marques-work marques-work mentioned this pull request Mar 29, 2019
Merged
@luiszimmermann luiszimmermann deleted the nss-fix branch April 1, 2019 12:19
@J0WI
Copy link
Contributor

J0WI commented Apr 1, 2019

This has been fixed upstream in https://git.alpinelinux.org/aports/commit/?h=3.9-stable&id=87ea954c9b7e200efbb52b4395dfc160fa4f0b55

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yosifkit yosifkit yosifkit approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@luiszimmermann @matsewe @tianon @J0WI @yosifkit