Skip to content

Build scripts cleanup#161

Merged
njsmith merged 3 commits intopypa:masterfrom
jcfr:build-scripts-cleanup
Apr 15, 2018
Merged

Build scripts cleanup#161
njsmith merged 3 commits intopypa:masterfrom
jcfr:build-scripts-cleanup

Conversation

@jcfr
Copy link
Contributor

@jcfr jcfr commented Mar 3, 2018

No description provided.

jcfr added a commit to dockbuild/dockbuild that referenced this pull request Mar 3, 2018
…rom centos:5

The differences between the two images are in the build.sh
script, They have been marked with comment block starting with
"[centosbuild]" and ending with "[/centosbuild]".

Remaining changes have been contributed upstream:
pypa/manylinux#161
pypa/manylinux#162

See https://github.com/pypa/manylinux
@jcfr
Copy link
Contributor Author

jcfr commented Mar 3, 2018

@njsmith In addition of few aesthetic cleanups, this PR has also the advantage of removing the curl tar.gz archive and saving few MBs.

It is ready to be merged.

@jcfr
Copy link
Contributor Author

jcfr commented Mar 28, 2018

@njsmith @matthew-brett What is the process to get this integrated ?

@njsmith
Copy link
Member

njsmith commented Apr 2, 2018

Are the changes here still correct now that #174 was merged? (They both have some mention of curl...)

@jcfr jcfr force-pushed the build-scripts-cleanup branch from 3147d5d to 24fc42d Compare April 2, 2018 08:52
@jcfr
Copy link
Contributor Author

jcfr commented Apr 2, 2018

Are the changes here still correct now that #174 was merged?

Indeed. Topic was rebased and CI is green.

@natefoo
Copy link
Member

natefoo commented Apr 2, 2018

With the switch back to the canonical archive, the old curl-* path should be correct (curl_* was for the debian archive).

The rest looks 👍 to me.

@jcfr jcfr force-pushed the build-scripts-cleanup branch from 24fc42d to 8efa4ef Compare April 2, 2018 14:15
@jcfr
Copy link
Contributor Author

jcfr commented Apr 2, 2018

the old curl-* path should be correct (curl_*

Indeed 👍 Updated topic pushed (from ✈️ )

@njsmith njsmith merged commit 4a20e4b into pypa:master Apr 15, 2018
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