* [PATCH v2 1/1] tools/libfsimage: Fix clean and distclean make targets
@ 2013-05-10 15:33 Daniel Kiper
2013-05-13 10:44 ` Ian Campbell
0 siblings, 1 reply; 2+ messages in thread
From: Daniel Kiper @ 2013-05-10 15:33 UTC (permalink / raw)
To: george.dunlap, ian.campbell, ian.jackson, stefano.stabellini, xen-devel
Cc: Daniel Kiper
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@oracle.com>
---
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 $< $@
--
1.7.10.4
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH v2 1/1] tools/libfsimage: Fix clean and distclean make targets
2013-05-10 15:33 [PATCH v2 1/1] tools/libfsimage: Fix clean and distclean make targets Daniel Kiper
@ 2013-05-13 10:44 ` Ian Campbell
0 siblings, 0 replies; 2+ messages in thread
From: Ian Campbell @ 2013-05-13 10:44 UTC (permalink / raw)
To: Daniel Kiper; +Cc: George Dunlap, xen-devel, Ian Jackson, Stefano Stabellini
On Fri, 2013-05-10 at 16:33 +0100, Daniel Kiper wrote:
> 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@oracle.com>
Acked-by: Ian Campbell <ian.campbell@citrix.com>
and pushed
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2013-05-13 10:44 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-05-10 15:33 [PATCH v2 1/1] tools/libfsimage: Fix clean and distclean make targets Daniel Kiper
2013-05-13 10:44 ` Ian Campbell
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.