-
Notifications
You must be signed in to change notification settings - Fork 38.7k
Fix "make check" on systems with qt and libxcb but no active DISPLAY #10117
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
Conversation
Allows partial execution of qt tests on systems that have x11 libraries but aren't running an X server or framebuffer.
If xvfb-run is not available, run an autoconf check to determine if instantiating a QApplication is possible, and if not, run the test with --skip-gui-tests. Fixes broken "make check" reported by Matt Corallo <[email protected]> in bitcoin#10110 Fixes bitcoin#10110
|
@ryanofsky This seems like overkill. How about theuni@b493819 ? Afaik, the minimal platform is available everywhere, and seems to be enough to make this work. (this needs a bit of extra configure logic though, so that we don't disable all of qt in the rare event that qminimal is missing). |
|
Interesting, I didn't know about the minimal platform. If you like, I can use your change as a starting point and try to add the needed configuration logic. I'd also like to keep optional support for the regular platforms, because it can be useful to show or interact with widgets while debugging the test. |
|
Sure. You can just make it minimal by default, and overridable to something else. Fwiw, you can use either "-platform minimal" or use the QT_QPA_PLATFORM env. The env was easier for a poc, but maybe you'll want to set the platform arg if there's not one set already. |
Fixes broken "make check" reported by Matt Corallo <[email protected]> in bitcoin#10110 Fix was suggested and initially implemented by Cory Fields <[email protected]> in bitcoin#10117 (comment)
|
Closing this in favor of #10142.
It seems easier to just require the minimal platform, unless there are cases where you think this isn't a reasonable requirement. Adding extra configure logic to make the check for |
Fixes broken "make check" reported by Matt Corallo <[email protected]> in bitcoin#10110 Fix was suggested and initially implemented by Cory Fields <[email protected]> in bitcoin#10117 (comment) Conflicts: src/qt/test/test_main.cpp
Fixes broken "make check" reported by Matt Corallo <[email protected]> in bitcoin/bitcoin#10110 Fix was suggested and initially implemented by Cory Fields <[email protected]> in bitcoin/bitcoin#10117 (comment)
Fixes broken "make check" reported by Matt Corallo <[email protected]> in bitcoin/bitcoin#10110 Fix was suggested and initially implemented by Cory Fields <[email protected]> in bitcoin/bitcoin#10117 (comment)
Fixes broken "make check" reported by Matt Corallo <[email protected]> in bitcoin/bitcoin#10110 Fix was suggested and initially implemented by Cory Fields <[email protected]> in bitcoin/bitcoin#10117 (comment)
Issue reported by @TheBlueMatt in #10110
This is an alternative to the fix in #10113. Instead of modifying test_bitcoin-qt to try to detect whether it's possible to run GUI tests with a check that is probably not reliable, it just adds a --skip-gui-tests option that can be controlled externally. Then, it uses autoconf checks to see if passing the option is necessary.