-
-
Notifications
You must be signed in to change notification settings - Fork 98
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
Add sdlinfo tool #628
Add sdlinfo tool #628
Conversation
Things that aren't included:
|
Real question, how testable are these routines in CI? |
Piper, asking the hard questions. In theory, this was written in a pretty testable way and we should be able to check their output pretty easily. But I have no idea how SDL2 behaves in a container on a headless server. |
Yup, that's my biggest concern. I feel like "no throwing an error" is all I actually care about, since the output is not guaranteed to be the same across different CI environments. |
At least some of the environment info is indicated in https://cirrus-ci.com/task/4754263726358528 |
@pathunstrom are those sufficient tests? |
ugh, I think I need a real live mac to debug those errors. |
Apparently haptics is the only system that requires being initialized in order to query it.
Yes, that's fine. If you're comfortable, I can start merging this. |
go ahead |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bors r+
Add a little tool that'll examine SDL and hopefully help track down any kind of "missing feature" problems.