Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Travis CI: Use pyenv on macOS #19

Closed
wants to merge 1 commit into from
Closed

Travis CI: Use pyenv on macOS #19

wants to merge 1 commit into from

Conversation

hroncok
Copy link
Member

@hroncok hroncok commented Jan 6, 2020

Fixes #17

hroncok
hroncok commented Jan 6, 2020
View reviewed changes
.travis.yml
# Install or update Python
- "brew install python3 || :"
- "brew upgrade python3"
- HOMEBREW_NO_INSTALL_CLEANUP=1 brew upgrade pyenv
Copy link
Member Author

Choose a reason for hiding this comment

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

this is optional, currently gives 3.7.6 over 3.7.5

@encukou
Copy link
Member

encukou commented Jan 8, 2020 •
edited

It looks like this will technically work, but it'll build CPython from source for every pull request. I don't want to do that to poor Travis.

I've added this to #18 to see what happens: b790b24.
I let Travis build it once and then restarted the job. Here's the output of the rebuild: https://travis-ci.org/pyvec/naucse_render/jobs/634235527#. The installation takes 3+ minutes.

@hroncok
Copy link
Member Author

hroncok commented Jan 8, 2020

is 3+ minutes good or bad?

@encukou
Copy link
Member

encukou commented Jan 8, 2020

Not too bad for compiling CPython! But I don't think compiling CPython shouldn't be part of the CI check here.

@hroncok
Copy link
Member Author

hroncok commented Jan 8, 2020

Too bad the cache does not contain the compiled interpreter :(

@encukou
Copy link
Member

encukou commented Jan 8, 2020

What's in the cache, anyway?

@hroncok
Copy link
Member Author

hroncok commented Jan 8, 2020

I guess it is the downloaded source code. But no idea really.

@encukou
Copy link
Member

encukou commented Feb 15, 2022

We're off Travis now.

@encukou encukou closed this Feb 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Test OS X & Windows on CI
2 participants
@hroncok @encukou