Skip to content
This repository has been archived by the owner on Jul 20, 2018. It is now read-only.

Fix GPG key retrieval and apply resilient key retrieval to slim Dockerfiles #88

Closed
wants to merge 2 commits into from
Closed

Fix GPG key retrieval and apply resilient key retrieval to slim Dockerfiles #88

wants to merge 2 commits into from

Conversation

jbhannah
Copy link
Contributor

@jbhannah jbhannah commented Sep 4, 2015

Docker Machine gave me the following when trying to build from the Dockerfiles with resilient GPG key retrieval added in #87:

/bin/sh: 1: Syntax error: end of file unexpected (expecting "done")
The command '/bin/sh -c set -ex   && for key in     9554F04D7259F04124DE6B476D5A82AC7E37093B     94AE36675C464D64BAFA68DD7434390BDBE9B9C5     0034A06D9D9B0064CE8ADF6BF1747F4AD2306D93     FD3A5288F042B6850C66B31F09FE44734EB7990E     71DCFD284A79C3B38668286BC97EC7A07EDE3FC1     DD8F2338BAE7501E3DD5AC78C273792F7D83545D   ; do     gpg --keyserver ha.pool.sks-keyservers.net --recv-keys "$key"   done' returned a non-zero code: 2

Fixed by adding a semicolon before done, and also applied resilient key retrieval to the slim Dockerfiles.

@jbhannah jbhannah mentioned this pull request Sep 4, 2015
@Starefossen
Copy link
Member

@hmalphettes any chance we can get this in before the PR to the official registry is merged?

@hmalphettes
Copy link
Member

Done in iojs: 5478095

hmalphettes added a commit to hmalphettes/official-images that referenced this pull request Sep 5, 2015
A regression was introduced here:
nodejs/docker-iojs#87

It is fixed here:
nodejs/docker-iojs#88
@hmalphettes
Copy link
Member

Added to the official-images PR: docker-library/official-images#1029
Let me know if I missed something.

@hmalphettes hmalphettes closed this Sep 5, 2015
@jbhannah jbhannah deleted the resilient-gpg-fix branch September 8, 2015 17:17
@jbhannah
Copy link
Contributor Author

jbhannah commented Sep 8, 2015

👍, LGTM. Thanks!

RichardScothern pushed a commit to RichardScothern/official-images that referenced this pull request Jun 14, 2016
A regression was introduced here:
nodejs/docker-iojs#87

It is fixed here:
nodejs/docker-iojs#88
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants