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.

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

Travis speedup #14

Merged
merged 2 commits into from
Aug 1, 2015
Merged

Travis speedup #14

merged 2 commits into from
Aug 1, 2015

Conversation

moreati
Copy link
Contributor

@moreati moreati commented Aug 1, 2015

Found the cause of your pytest failure: PyPy bundles an old and customised version of pytest

- Use latest pip which defaults to downloading/building/caching wheels
- Save pip cache between builds
- Declare/install all requirements in a single place
@snim2
Copy link
Member

snim2 commented Aug 1, 2015

Awesome, many thanks!

snim2 added a commit that referenced this pull request Aug 1, 2015
@snim2 snim2 merged commit 01ecfe3 into futurecore:master Aug 1, 2015
@moreati
Copy link
Contributor Author

moreati commented Aug 1, 2015

Spoke too soon, from https://travis-ci.org/futurecore/revelation/builds/73694067

The command "python -m doctest scripts/diff_trace.py" exited with 0.
0.35s$ test "$TRANSLATE" == "true" && ${HOME}/pypy/rpython/bin/rpython --opt=jit epiphany/sim.py || true
Traceback (most recent call last):
  File "app_main.py", line 75, in run_toplevel
  File "/home/travis/pypy/rpython/bin/rpython", line 13, in <module>
    from rpython.translator.goal.translate import main
  File "/home/travis/pypy/rpython/translator/goal/translate.py", line 10, in <module>
    import py
ImportError: No module named py

@snim2
Copy link
Member

snim2 commented Aug 1, 2015

The translation isn't so important at the moment. I'll ask the pypy people how they deal with this stuff in their own repos. Maybe they don't install py.test from pip.

@moreati
Copy link
Contributor Author

moreati commented Aug 1, 2015

Okay, note that Travis install pytest themselves and pytest requires py - so you get some version regardless of pip or PyPy

@snim2
Copy link
Member

snim2 commented Aug 1, 2015

Good point.

@moreati
Copy link
Contributor Author

moreati commented Aug 1, 2015

There's https://pypi.python.org/pypi/rpython/ - nothing bundled, depends on pytest, last release Apr 2015. However it does say

Warning
This is an experimental release of a randomly chosen, untested version of RPython. Packaging issues are likely, feedback is welcome.

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