Make CI fail if interactive toolkits can't be tested#11497
Conversation
| python -mpip install --upgrade $PRE pgi | ||
| python -c 'import pgi as gi; gi.require_version("Gtk", "3.0"); from pgi.repository import Gtk' | ||
| fi | ||
| python -c 'import PyQt5.QtCore' |
There was a problem hiding this comment.
These still need the echo bit to make sure the build doesn't fail if they aren't importable.
There was a problem hiding this comment.
Actually the point is exactly to fail when they are not importable, instead of having to figure this out manually every time this happens.
|
I have a hack solution to fix PyQt5 5.11 here, but I need to turn it into something reasonable. |
|
See #11500. |
|
PyQt5.11 has been withdrawn (https://riverbankcomputing.com/pipermail/pyqt/2018-June/040459.html), so I repurposed the PR to just move GUI deps to requirement files too and fail the CI when they fail to install properly. |
|
So it appears that the interactive tests are failing with wx+OSX. Any OSX specialist wants to have a look at this (can you repro locally?), or should we just skip them? |
|
I'm a little confused where we stand here. Right now none of the Travis CI tests are passing because of QT5 issues. Is this fix, correct, or #11500? or are they complimentary? |
anntzer
changed the title
|
See #11500 (comment); as for this PR the scope has changed and I renamed it accordingly. |
| # pgi is only installable on Linux | ||
| pyqt5 | ||
| # wxpython 4.0.2 is broken on OSX, see https://groups.google.com/forum/#!topic/wxpython-dev/lNvdniiXhdY | ||
| wxpython!=4.0.2 |
There was a problem hiding this comment.
Looks like 4.0.3 is also failing now?
There was a problem hiding this comment.
let's try excluding it too...
There was a problem hiding this comment.
Fails to build faster! Actually I'm not clear why the mac build is trying to use 3.7....
Also make it a failure to fail to install the GUI toolkits for testing, to avoid missing new failures there.
|
Force-pushed a change to pin python to 3.6 for OS X. It was trying to use 3.7. |
|
Well, that didn't work either. Sorry, you should revert my commits... |
|
but now it's failing on something different (pyqt5). I'll try something later. |
|
This looks to be broken, but feel free to re-open if I'm wrong... |
|
I think some variant of this PR should still go in, but let's do this after the current qt breakage is fixed :) |
5 participants
PyQt5.11 appears to break the build (I can reproduce the failurelocally).
Also make it a failure to fail to install the GUI toolkits for testing,
to avoid missing new failures there.
Also supersedes #11494.
PR Summary
PR Checklist