[Scummvm-devel] Event Recorder Dependencies Breaking Tests
D G Turner
d.g.turner at ntlworld.com
Sat Jul 6 06:21:34 CEST 2013
Eugene,
I have looked at the reason for the "make test" failure since the
Event Recorder was merged. This is due to the new and undesired
dependency between the common and GUI code introduced by this.
Though removing all of this requirement is a more complex design
question, I have worked out that it is possible to restore the
tests functionality for now.
This will require the attached patch to common/system.cpp to be
applied. It adds #ifdef ENABLE_EVENTRECORDER guards which are
currently not present in this file, but are very similar to those
present in various other files including base/main.cpp.
This will then truly remove the dependency when the event recorder
is disabled, rather than leaving some linkages in system.cpp.
I can then fix the tests on buildbot by adding "--disable-
eventrecorder" to the configure options passed to the debian-x86
native build.
As this is a fairly trivial patch, I didn't think it warranted a
Pull Request, but I didn't want to cause another argument with the
Core Team if I applied this fix directly, so am sending it to the
list for comment and review.
I am aware that this could be considered a "hack", though my view is
that those guards should be present in that file in any case and this
solution has the advantage of restoring the tests functionality now,
rather that leaving these broken (which would leave the common code
open to developing bugs which are not picked up until much later).
Any views on this? If there are no objections, I will apply this
patch in the next few days and make the requisite change to buildbot.
Thanks,
David Turner
-------------- next part --------------
A non-text attachment was scrubbed...
Name: guard_event_recorder_in_system_cpp.patch
Type: text/x-patch
Size: 663 bytes
Desc: not available
URL: <http://lists.scummvm.org/pipermail/scummvm-devel/attachments/20130706/2ae2e9b8/attachment.bin>
More information about the Scummvm-devel
mailing list