On Thu, Jan 21, 2016 at 07:33:11AM -0500, Frediano Ziglio wrote:
> > 
> > The next commit will introduce a test for log messages. As
> > libspice-common.la behaviour varies depending on whether
> > SPICE_DISABLE_ASSERT was defined at compile-time, this test will also
> > take into account this preprocessor define.
> > We are more likely to get a consistent build (SPICE_DISABLE_ASSERT being
> > the same when building libspice-common.la and the test) if both are
> > built at the same time.
> > 
> > This commit changes that, tests are now built (but not run) at 'make'
> > time rather than 'make check' time.
> 
> Nack!
> This check marshalling code, should have nothing to do with logging.
> If has the test should be fixed.

This commit changes when the tests are built (make VS make check).

This commit has nothing to do with logging except for the log
explaining the rationale for building at make time rather than make
check time.

Christophe

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Spice-devel mailing list
Spice-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/spice-devel

Reply via email to