[Qemu-devel] [PATCH] tests: remove .SECONDARY special target

[Qemu-devel] [PATCH] tests: remove .SECONDARY special target

[Paolo Bonzini]

The special target should not be needed anymore, and caused (perhaps
due to a Make bug) a failure with "make -j2".  In any case, the
main makefile is a better place for such special targets rather
than an included makefile.

Reported-by: Luiz Capitulino <lcapitulino@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
---
	Anthony, this was taken from your patches, do you remember
        why it's needed?  I changed the rules to use the GNU Make
        "LIST: TARGET-PATTERN: SOURCE-PATTERN" syntax so there
        should be no implicit rules anymore and .SECONDARY should
        not be needed.

 tests/Makefile |    2 --
 1 files changed, 0 insertions(+), 2 deletions(-)

diff --git a/tests/Makefile b/tests/Makefile
index 17f6ece..25a972a 100644
--- a/tests/Makefile
+++ b/tests/Makefile
@@ -74,8 +74,6 @@ check-help:
 	@echo "Default options are -k and (for make V=1) --verbose; they can be"
 	@echo "changed with variable GTESTER_OPTIONS."
 
-.SECONDARY:
-
 SPEED = quick
 GTESTER_OPTIONS = -k $(if $(V),--verbose,-q)
 
-- 
1.7.9.1

Re: [Qemu-devel] [PATCH] tests: remove .SECONDARY special target

[Luiz Capitulino]

On Sat, 31 Mar 2012 12:50:21 +0200
Paolo Bonzini <pbonzini@redhat.com> wrote:

> The special target should not be needed anymore, and caused (perhaps
> due to a Make bug) a failure with "make -j2".  In any case, the
> main makefile is a better place for such special targets rather
> than an included makefile.
> 
> Reported-by: Luiz Capitulino <lcapitulino@redhat.com>
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>

Tested-by: Luiz Capitulino <lcapitulino@redhat.com>

Nice to see it building again.

> ---
> 	Anthony, this was taken from your patches, do you remember
>         why it's needed?  I changed the rules to use the GNU Make
>         "LIST: TARGET-PATTERN: SOURCE-PATTERN" syntax so there
>         should be no implicit rules anymore and .SECONDARY should
>         not be needed.
> 
>  tests/Makefile |    2 --
>  1 files changed, 0 insertions(+), 2 deletions(-)
> 
> diff --git a/tests/Makefile b/tests/Makefile
> index 17f6ece..25a972a 100644
> --- a/tests/Makefile
> +++ b/tests/Makefile
> @@ -74,8 +74,6 @@ check-help:
>  	@echo "Default options are -k and (for make V=1) --verbose; they can be"
>  	@echo "changed with variable GTESTER_OPTIONS."
>  
> -.SECONDARY:
> -
>  SPEED = quick
>  GTESTER_OPTIONS = -k $(if $(V),--verbose,-q)
>