Skip to content

Add php-7.0.0alpha2 (Issue #108) #109

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

Conversation

marceloandrader
Copy link
Contributor

I've modified update.sh to support versions without gpg signature verification

The dockerfile on 7.0.0alpha2 directory was modified a bit to download the file from another location without gpg verfication, this will be fixed when 7 goes stable and we have the .asc file to verify.

the gpg key from php 7 was found on http://php.net/gpg-keys.php

generate-stackbrew-library.sh works without issues.

@@ -3,6 +3,7 @@ set -e

declare -A gpgKeys
gpgKeys=(
[7.0.0alpha2]='1A4E8B7277C42E53DBA9C7B9BCAA30EA9C0D5763'
Copy link
Member

Choose a reason for hiding this comment

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

Can you please rename the folder to just be 7.0?

@marceloandrader
Copy link
Contributor Author

@tianon I've updated the code as you suggested.

RUN set -xe \
&& for key in $GPG_KEYS; do \
gpg --keyserver ha.pool.sks-keyservers.net --recv-keys "$key"; \
done
Copy link
Member

Choose a reason for hiding this comment

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

I would just drop the gpg key env and downloading, since it is unused and will probably change on the release anyway.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@yosifkit this code will be kept when the 7.0 goes to production in that case we will make it compatible with 5.6 and verify the download.

Copy link
Member

Choose a reason for hiding this comment

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

As you point out, this file will need to be edited to actually do the gpg verify when 7.0 is released. Having this key serves no purpose in the image and will likely be a different key on release anyway.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@yosifkit Ok removed

@mykiwi
Copy link
Contributor

mykiwi commented Jul 8, 2015

You should replace alpha2 with beta1

@tianon tianon mentioned this pull request Jul 8, 2015
@tianon
Copy link
Member

tianon commented Jul 8, 2015

I've rebased/carried this PR over in #113. 👍

@yosifkit yosifkit closed this in #113 Jul 8, 2015
@marceloandrader
Copy link
Contributor Author

Ok thanks @tianon

@tianon
Copy link
Member

tianon commented Jul 8, 2015 via email

tianon added a commit to infosiftr/stackbrew that referenced this pull request Jul 8, 2015
- `golang`: 1.5beta1
- `mysql`: time zones (docker-library/mysql#80)
- `php`: 7.0.0beta1 (docker-library/php#113, docker-library/php#109)
- `postgres`: 9.5-alpha1 (docker-library/postgres#70), jessie (docker-library/postgres#71)
RichardScothern pushed a commit to RichardScothern/official-images that referenced this pull request Jun 14, 2016
- `golang`: 1.5beta1
- `mysql`: time zones (docker-library/mysql#80)
- `php`: 7.0.0beta1 (docker-library/php#113, docker-library/php#109)
- `postgres`: 9.5-alpha1 (docker-library/postgres#70), jessie (docker-library/postgres#71)
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.

4 participants