Skip to content

CI: 3.8 build #28730

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

Merged
merged 48 commits into from
Oct 15, 2019
Merged

CI: 3.8 build #28730

merged 48 commits into from
Oct 15, 2019

Conversation

jbrockmendel
Copy link
Member

@jbrockmendel jbrockmendel commented Oct 1, 2019

Travis config is a mystery to me, so this is trial-and-error until I get the hang of it.

Closes #26626

Copy link
Member

@WillAyd WillAyd left a comment

Choose a reason for hiding this comment

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

I think the failure is a result of changing the dist. This link might have the answer:

https://docs.travis-ci.com/user/trusty-to-xenial-migration-guide#1-services-support

I think would need to add that key for both mysql and psql

@jreback
Copy link
Contributor

jreback commented Oct 2, 2019

see #27266 as well here

@jbrockmendel
Copy link
Member Author

For a moment I was amazed that the last "restore everything disabled during debugging" commit came back green.

Then I saw that I had mangled the Travis config so badly that Travis didn't run at all.

@alimcmaster1 alimcmaster1 added the CI Continuous Integration label Oct 7, 2019
@TomAugspurger
Copy link
Contributor

FYI, on the NumPy mailing list they announced that they built 3.8 wheels. Looks like https://github.com/MacPython/numpy-wheels/blob/master/.travis.yml

Maybe MacPython/numpy-wheels@9d8a0ad + an update of multibuild in the pandas-wheels repo would do the trick.

@jbrockmendel
Copy link
Member Author

Maybe MacPython/numpy-wheels@9d8a0ad + an update of multibuild in the pandas-wheels repo would do the trick.

Does this replace the approach being taken in this PR? Or supplement?

@jbrockmendel
Copy link
Member Author

Reverted to use git clone, with comment about possible pip alternative

- sleep 3

script:
- echo "script start"
- source activate pandas-dev
- echo "$JOB"
- if [ "$JOB" != "3.8-dev" ]; then source activate pandas-dev; fi
Copy link
Contributor

Choose a reason for hiding this comment

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

why is this not also called pandas-dev (the env name)?

Copy link
Member Author

Choose a reason for hiding this comment

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

You mean $JOB? none of the other $JOB values have pandas-dev in the name

@@ -0,0 +1,25 @@
#!/bin/bash -e
# Special build for python3.8 until numpy puts its own wheels up

Copy link
Contributor

Choose a reason for hiding this comment

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

you should create a bare conda env here

Copy link
Member Author

Choose a reason for hiding this comment

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

how come?

Copy link
Contributor

Choose a reason for hiding this comment

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

3.8 should not be very different than a regular setup

Copy link
Contributor

Choose a reason for hiding this comment

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

It has to be different until there are 3.8 packages (python especially, but also our dependencies).

I think Brock has about the minimal difference we can get right now.

@TomAugspurger
Copy link
Contributor

@jreback thoughts on #28730 (comment)?

3.8 is out now, so I'd like to get this in and 0.25.2 released as soon as possible (ideally by sometime tomorrow).

@jreback jreback added this to the 0.25.2 milestone Oct 14, 2019
@TomAugspurger
Copy link
Contributor

#29001 for followup.

Thanks @jbrockmendel.

@TomAugspurger TomAugspurger merged commit 80412cc into pandas-dev:master Oct 15, 2019
meeseeksmachine pushed a commit to meeseeksmachine/pandas that referenced this pull request Oct 15, 2019
TomAugspurger pushed a commit that referenced this pull request Oct 15, 2019
@jbrockmendel jbrockmendel deleted the build38 branch October 15, 2019 15:27
proost pushed a commit to proost/pandas that referenced this pull request Dec 19, 2019
proost pushed a commit to proost/pandas that referenced this pull request Dec 19, 2019
bongolegend pushed a commit to bongolegend/pandas that referenced this pull request Jan 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous Integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Setup python 3.8 builds
5 participants