Message ID | 8736ibtogp.fsf@cbaines.net |
---|---|
Headers | show |
Hi Chris, Christopher Baines <mail@cbaines.net> skribis: > This patch series adds the Unknown Horizons game, and a couple of it's > dependencies. Looks all good to me! :-) Ludo’.
>> This patch series adds the Unknown Horizons game, and a couple of it's >> dependencies. > > Looks all good to me! :-) fifengine and unknown-horizons do have tests, though. Could you try packaging them?
> The unknown-horizons tests seem fine, but the fifengine tests don't all > seem to run, so I've left them disabled. I don't really get what the > test runner is doing. Neither did I manage to completely figure out the fifengine tests. Hence my incomplete patches. Please feel free to push without the tests or with any other appropriate comments.
Arun Isaac <arunisaac@systemreboot.net> writes: >> The unknown-horizons tests seem fine, but the fifengine tests don't all >> seem to run, so I've left them disabled. I don't really get what the >> test runner is doing. > > Neither did I manage to completely figure out the fifengine tests. Hence > my incomplete patches. Please feel free to push without the tests or > with any other appropriate comments. Ok, cool. I've pushed the patches now as a1c1943b7c49dfb6aef9a98b2cbce0d31cbed08b. Now I just need to work out how to play the game! Thanks, Chris