qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: "Andreas Färber" <afaerber@suse.de>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: Paolo Bonzini <pbonzini@redhat.com>,
	qemu-devel@nongnu.org, Anthony Liguori <anthony@codemonkey.ws>
Subject: Re: [Qemu-devel] [PATCH] Makefile.target: Update clean command to clean hw/ directory
Date: Wed, 20 Jun 2012 15:58:53 +0200	[thread overview]
Message-ID: <4FE1D71D.10300@suse.de> (raw)
In-Reply-To: <1340200496-17126-1-git-send-email-peter.maydell@linaro.org>

Am 20.06.2012 15:54, schrieb Peter Maydell:
> Now we create object files in a hierarchy under hw/, so the
> 'clean' target must also updated to delete those object files.

"be updated"

Other than that and the -f looks good. Thanks.

/-F

> Rather than using a manual list of subdirectories which will
> easily drift out of date, we just delete all .o and .d files
> in the target directory hierarchy.
> 
> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
> ---
>  Makefile.target |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/Makefile.target b/Makefile.target
> index 550d889..686ba8d 100644
> --- a/Makefile.target
> +++ b/Makefile.target
> @@ -193,8 +193,8 @@ qmp-commands-old.h: $(SRC_PATH)/qmp-commands.hx
>  	$(call quiet-command,sh $(SRC_PATH)/scripts/hxtool -h < $< > $@,"  GEN   $(TARGET_DIR)$@")
>  
>  clean:
> -	rm -f *.o *.a *~ $(PROGS) nwfpe/*.o fpu/*.o
> -	rm -f *.d */*.d tcg/*.o ide/*.o 9pfs/*.o kvm/*.o
> +	rm -f *.a *~ $(PROGS)
> +	rm $(shell find . -name '*.[od]')
>  	rm -f hmp-commands.h qmp-commands-old.h gdbstub-xml.c
>  ifdef CONFIG_TRACE_SYSTEMTAP
>  	rm -f *.stp


-- 
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg

  parent reply	other threads:[~2012-06-20 13:59 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-06-20 13:54 [Qemu-devel] [PATCH] Makefile.target: Update clean command to clean hw/ directory Peter Maydell
2012-06-20 13:56 ` Peter Maydell
2012-06-20 13:58 ` Andreas Färber [this message]
  -- strict thread matches above, loose matches on Subject: below --
2012-06-20 17:02 Peter Maydell
2012-06-20 17:13 ` Peter Maydell

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4FE1D71D.10300@suse.de \
    --to=afaerber@suse.de \
    --cc=anthony@codemonkey.ws \
    --cc=pbonzini@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).