libunity

Merge lp://qastaging/~ballogy/libunity/make-tests-optional into lp://qastaging/libunity

make-tests-optional
Merge into trunk

Proposed by Balló György on 2011-11-27

Status:	Needs review
Proposed branch:	lp://qastaging/~ballogy/libunity/make-tests-optional
Merge into:	lp://qastaging/libunity
Diff against target:	44 lines (+14/-1) 2 files modified Makefile.am (+4/-1) configure.ac (+10/-0)
To merge this branch:	bzr merge lp://qastaging/~ballogy/libunity/make-tests-optional
Related bugs:	Link a bug report

Reviewer	Review Type	Date Requested	Status
James Henstridge		2011-11-27	Needs Information on 2013-05-28
Review via email: mp+83501@code.qastaging.launchpad.net

Description of the change

This change makes it possible to disable building tests with '--disable-tests' configure switch.

Revision history for this message

Mikkel Kamstrup Erlandsen (kamstrup) wrote on 2011-11-28:

Hey György. Thanks for this. We're just going through some big changes
in the libunity build setup, so we'll get back to this one once the
bigger stuff lands. Sorry to keep you waiting.

Revision history for this message

James Henstridge (jamesh) wrote on 2013-05-28:

Is this MP still relevant? With a current checkout, if I run "make" followed by "make check", I see the Vala tests being compiled on the second invocation.

Does the configure flag add much benefit on top of this?

review: Needs Information