[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] Makefile: add fsdev/*.{o,d} to clean
From: |
Hidetoshi Seto |
Subject: |
Re: [Qemu-devel] [PATCH] Makefile: add fsdev/*.{o,d} to clean |
Date: |
Fri, 02 Jul 2010 15:27:57 +0900 |
User-agent: |
Mozilla/5.0 (Windows; U; Windows NT 6.0; ja; rv:1.9.2.4) Gecko/20100608 Thunderbird/3.1 |
(2010/07/02 4:48), Blue Swirl wrote:
> 2010/7/1 Hidetoshi Seto <address@hidden>:
>> There were fsdev/qemu-fsdev.{o,d} not removed at "make clean".
>>
>> Signed-off-by: Hidetoshi Seto <address@hidden>
>> ---
>> Makefile | 2 +-
>> 1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/Makefile b/Makefile
>> index 560eac6..ce5f0e6 100644
>> --- a/Makefile
>> +++ b/Makefile
>> @@ -159,7 +159,7 @@ clean:
>> # avoid old build problems by removing potentially incorrect old files
>> rm -f config.mak op-i386.h opc-i386.h gen-op-i386.h op-arm.h
>> opc-arm.h gen-op-arm.h
>> rm -f *.o *.d *.a $(TOOLS) TAGS cscope.* *.pod *~ */*~
>> - rm -f slirp/*.o slirp/*.d audio/*.o audio/*.d block/*.o block/*.d
>> net/*.o net/*.d
>> + rm -f {slirp,audio,block,net,fsdev}/*.{o,d}
>
> Not every shell out there support {}:
> dash -c 'echo {a,b}*.c'
> {a,b}*.c
>
> bash -c 'echo {a,b}*.c'
> acl.c aes.c aio.c alpha-dis.c arch_init.c arm-dis.c arm-semi.c async.c
> balloon.c block-migration.c block.c blockdev.c bt-host.c bt-vhci.c
> buffered_file.c
You are right, but I found that Makefile already has:
169 distclean: clean
(snip)
174 rm -f
qemu-{doc,tech}.{info,aux,cp,dvi,fn,info,ky,log,pdf,pg,toc,tp,vr}
So then is it better to rewrite this line not to use {} ...?
Thanks,
S.Seto