Skip to content

3.6/alpine3.7: Build nis module now that 3.6.5 is patched#274

Merged
yosifkit merged 1 commit intodocker-library:masterfrom
JayH5:py3.6-alpine3.7-nis
Mar 30, 2018
Merged

3.6/alpine3.7: Build nis module now that 3.6.5 is patched#274
yosifkit merged 1 commit intodocker-library:masterfrom
JayH5:py3.6-alpine3.7-nis

Conversation

@JayH5
Copy link
Copy Markdown
Contributor

@JayH5 JayH5 commented Mar 29, 2018

Continuation of #266.

Note also there's a Python 3.7.0b3 out now...guess the bot hasn't got to it yet.

# https://bugs.python.org/issue32521
# TODO: Remove Python version check once 2.7 and 3.6 have the patch
if [[ "$variant" == alpine* ]] && [[ "$variant" != alpine3.7 || "$version" != 3.7* ]]; then
if [[ "$variant" == alpine* ]] && [[ "$variant" != alpine3.7 ]]; then
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Won't this also leave the libraries on python:2.7-alpine3.7? Is that version fixed too?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I did think so when I wrote this the first time, but the Python 2.7 Dockerfiles actually aren’t templated.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

You are correct; I forgot about these lines

python/update.sh

Lines 135 to 136 in 33b16b4

if [[ "$version" == 2.* ]]; then
echo " TODO: vimdiff 3.6/$v/Dockerfile $version/$v/Dockerfile"

@yosifkit yosifkit merged commit 3715773 into docker-library:master Mar 30, 2018
yosifkit added a commit to infosiftr/stackbrew that referenced this pull request Apr 3, 2018
- `docker` update to `18.04.0-ce-rc1`
- `ghost` update to `1.22.1`
- `golang` docker-library/golang#215
- `openjdk` minor Debian openjdk bumps
- `php` bump to `7.2.4`, `7.1.16`, `7.0.29`, and `5.6.35`
- `python` docker-library/python#274
- `tomcat` openssl `1.1.0f-3+deb9u2`
davidl-zend pushed a commit to davidl-zend/official-images that referenced this pull request Apr 17, 2018
- `docker` update to `18.04.0-ce-rc1`
- `ghost` update to `1.22.1`
- `golang` docker-library/golang#215
- `openjdk` minor Debian openjdk bumps
- `php` bump to `7.2.4`, `7.1.16`, `7.0.29`, and `5.6.35`
- `python` docker-library/python#274
- `tomcat` openssl `1.1.0f-3+deb9u2`
davidl-zend pushed a commit to davidl-zend/official-images that referenced this pull request Apr 17, 2018
- `docker` update to `18.04.0-ce-rc1`
- `ghost` update to `1.22.1`
- `golang` docker-library/golang#215
- `openjdk` minor Debian openjdk bumps
- `php` bump to `7.2.4`, `7.1.16`, `7.0.29`, and `5.6.35`
- `python` docker-library/python#274
- `tomcat` openssl `1.1.0f-3+deb9u2`
tao12345666333 pushed a commit to tao12345666333/python that referenced this pull request Jun 28, 2018
3.6/alpine3.7: Build nis module now that 3.6.5 is patched
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.

2 participants