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

test_convert fails with --no-display flag #626

Closed
tobihan opened this issue Apr 26, 2016 · 3 comments
Closed

test_convert fails with --no-display flag #626

tobihan opened this issue Apr 26, 2016 · 3 comments

Comments

@tobihan
Copy link
Contributor

tobihan commented Apr 26, 2016

Hi,

when building the Debian package for Allegro 5.2, the test suite is run with the flag --no-display. In this case the test test_convert.ini fails with the error:

test_driver:.../src/drawing.c:29: al_clear_to_color: Assertion `target' failed.

Could you please have a look?

@SiegeLord
Copy link
Member

test_convert.ini only works with a display. I suggest removing it in the rules file (that's what I did for the Ubuntu packages).

@tobihan
Copy link
Contributor Author

tobihan commented Apr 27, 2016

Then wouldn't it be nicer if the flag --no-display would disable these tests? It seems that's the purpose of this flag.

@SiegeLord
Copy link
Member

Yeah, that's a good idea. I can annotate the tests which require a display and then this would work automatically.

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

No branches or pull requests

2 participants