[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-trivial] [Qemu-devel] [PATCH] Makefile: Properly order build t

From: Markus Armbruster
Subject: Re: [Qemu-trivial] [Qemu-devel] [PATCH] Makefile: Properly order build targets 'all' and 'check'
Date: Thu, 25 Jun 2015 09:08:29 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.3 (gnu/linux)

Stefan Weil <address@hidden> writes:

> Am 23.06.2015 um 19:31 schrieb Peter Maydell:
>> On 23 June 2015 at 14:35, Michal Privoznik <address@hidden> wrote:
>>> Yeah, that could work too. For some reason I thought that having it
>>> there would result in making 'all' just under tests/. But Now that I
>>> tried it out it works just nicely.
>> Have you tested both "build in the source tree" and "build in
>> a separate directory from the source tree", by the way?
>> thanks
>> -- PMM
> Both will work, as the modification only adds a dependency.
> Do we care that running "make check" will take longer with this
> patch? Make needs some time to check all dependencies for
> "all", even if nothing has to be done.

If this bothers us, we could try making it an order-only prerequisite:

check: | all


> I feel a little bit uneasy with something depending on all.
> Maybe some day we'll want to include check in the default
> build. Then all would depend on check which depends on

I agree that depending on the default goal (here: all) isn't nice.

> all which depends on check and so on. An intermediate
> make target could solve that:
> all: full-build
> check: full-build
> full-build: $(DOCS) $(TOOLS) $(HELPERS-y) recurse-all modules

reply via email to

[Prev in Thread] Current Thread [Next in Thread]