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

[Xen-changelog] [xen master] tools/libfsimage: Fix clean and distclean make targets



commit 667d8a84b244d02e9c6a2d02d6a02fc90c2efb4e
Author:     Daniel Kiper <daniel.kiper@xxxxxxxxxx>
AuthorDate: Fri May 10 17:33:54 2013 +0200
Commit:     Ian Campbell <ian.campbell@xxxxxxxxxx>
CommitDate: Mon May 13 10:57:17 2013 +0100

    tools/libfsimage: Fix clean and distclean make targets
    
    If there is a single colon for a given target and the target
    is redefined in another place (e.g. in included file) then
    make executes only new target and displays following warning:
    
    Makefile:35: warning: overriding commands for target `clean'
    tools/libfsimage/common/../../../tools/libfsimage/Rules.mk:25:
    warning: ignoring old commands for target `clean'
    
    To cope with that issue define all required targets as double-colon
    rules. Additionally, remove some redundant stuff.
    
    Signed-off-by: Daniel Kiper <daniel.kiper@xxxxxxxxxx>
    Acked-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
---
 tools/libfsimage/Rules.mk        |    2 +-
 tools/libfsimage/common/Makefile |    4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/tools/libfsimage/Rules.mk b/tools/libfsimage/Rules.mk
index e4b2a91..8a23655 100644
--- a/tools/libfsimage/Rules.mk
+++ b/tools/libfsimage/Rules.mk
@@ -21,7 +21,7 @@ fs-install: fs-all
 $(FSLIB): $(PIC_OBJS)
        $(CC) $(LDFLAGS) $(SHLIB_LDFLAGS) -o $@ $^ -lfsimage $(FS_LIBDEPS) 
$(APPEND_LDFLAGS)
 
-clean distclean:
+clean distclean::
        rm -f $(PIC_OBJS) $(FSLIB) $(DEPS)
 
 -include $(DEPS)
diff --git a/tools/libfsimage/common/Makefile b/tools/libfsimage/common/Makefile
index f0ee3ab..cbd60b4 100644
--- a/tools/libfsimage/common/Makefile
+++ b/tools/libfsimage/common/Makefile
@@ -31,8 +31,8 @@ install: all
        $(INSTALL_DATA) fsimage_plugin.h $(DESTDIR)$(INCLUDEDIR)
        $(INSTALL_DATA) fsimage_grub.h $(DESTDIR)$(INCLUDEDIR)
 
-clean distclean:
-       rm -f $(PIC_OBJS) $(LIB) $(DEPS)
+clean distclean::
+       rm -f $(LIB)
 
 libfsimage.so: libfsimage.so.$(MAJOR)
        ln -sf $< $@
--
generated by git-patchbot for /home/xen/git/xen.git#master

_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxx
http://lists.xensource.com/xen-changelog


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.