[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] Makefile: Properly order build targets 'all' an
From: |
Markus Armbruster |
Subject: |
Re: [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
https://www.gnu.org/software/make/manual/html_node/Prerequisite-Types.html
> 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
- [Qemu-devel] [PATCH] Makefile: Properly order build targets 'all' and 'check', Michal Privoznik, 2015/06/23
- Re: [Qemu-devel] [PATCH] Makefile: Properly order build targets 'all' and 'check', Peter Maydell, 2015/06/23
- Re: [Qemu-devel] [PATCH] Makefile: Properly order build targets 'all' and 'check', Michal Privoznik, 2015/06/23
- Re: [Qemu-devel] [PATCH] Makefile: Properly order build targets 'all' and 'check', Thomas Huth, 2015/06/23
- Re: [Qemu-devel] [PATCH] Makefile: Properly order build targets 'all' and 'check', Michal Privoznik, 2015/06/23
- Re: [Qemu-devel] [Qemu-trivial] [PATCH] Makefile: Properly order build targets 'all' and 'check', Michael Tokarev, 2015/06/23
- Re: [Qemu-devel] [PATCH] Makefile: Properly order build targets 'all' and 'check', Peter Maydell, 2015/06/23
- Re: [Qemu-devel] [PATCH] Makefile: Properly order build targets 'all' and 'check', Stefan Weil, 2015/06/23
- Re: [Qemu-devel] [PATCH] Makefile: Properly order build targets 'all' and 'check',
Markus Armbruster <=
- Re: [Qemu-devel] [PATCH] Makefile: Properly order build targets 'all' and 'check', Michal Privoznik, 2015/06/25