Date: prev next · Thread: first prev next last
2011 Archives by date, by thread · List index


Hi Bjoern,

On 2011-03-21 at 18:34 +0100, Bjoern Michaelsen wrote:

We clearly want to run unittest by default. It might make sense to
rename the current "all" target to "buildonly" and "allandcheck" to
"all" to be more explicit. The GNU make conventions are unclear if
checks should be run by the default target (However, they are clear on
that the default target should be "all" -- which we currently violate):

 http://www.gnu.org/software/make/manual/make.html#Standard-Targets

Im open for opinions on this.

I'd default to building without running checks when you issue make in
the module (the developer running the make knows best whether the change
is just adding a debug output, or does the "Real Thing"), and build with
checks when you rebuild the entire LibreOffice - that way, even if you
forget to do your 'make check', your changes are high that you spot the
breakage, or (in the worst case) the tinderboxes uncover that.

Wrt. the naming, I'd do 'all' build only, and 'check' depending on
'all', ie. the 'allandcheck' meaning.

Regards,
Kendy


Context


Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.