]> xenbits.xensource.com Git - people/tklengyel/xen.git/commitdiff
tools/xenpaging: Rework makefile
authorAnthony PERARD <anthony.perard@citrix.com>
Mon, 15 Aug 2022 06:55:23 +0000 (08:55 +0200)
committerJan Beulich <jbeulich@suse.com>
Mon, 15 Aug 2022 06:55:23 +0000 (08:55 +0200)
- Rename $(SRCS) to $(OBJS-y), we don't need to collect sources.
- Rename $(IBINS) to $(TARGETS)
- Stop cleaning "xen" and non-set variable $(LIB).

Signed-off-by: Anthony PERARD <anthony.perard@citrix.com>
Reviewed-by: Luca Fancellu <luca.fancellu@arm.com>
tools/xenpaging/Makefile

index 04743b335ce1251e86898528b5320e9f61879068..e2ed9eaa3fcc366bce5265ff3b4eb36ea7082bd8 100644 (file)
@@ -5,33 +5,33 @@ CFLAGS += $(CFLAGS_libxentoollog) $(CFLAGS_libxenevtchn) $(CFLAGS_libxenctrl) $(
 LDLIBS += $(LDLIBS_libxentoollog) $(LDLIBS_libxenevtchn) $(LDLIBS_libxenctrl) $(LDLIBS_libxenstore) $(PTHREAD_LIBS)
 LDFLAGS += $(PTHREAD_LDFLAGS)
 
-POLICY    = default
+POLICY   := default
 
-SRC      :=
-SRCS     += file_ops.c xenpaging.c policy_$(POLICY).c
-SRCS     += pagein.c
+OBJS-y   := file_ops.o
+OBJS-y   += xenpaging.o
+OBJS-y   += policy_$(POLICY).o
+OBJS-y   += pagein.o
 
 CFLAGS   += -Werror
 CFLAGS   += -Wno-unused
 
-OBJS     = $(SRCS:.c=.o)
-IBINS    = xenpaging
+TARGETS := xenpaging
 
-all: $(IBINS)
+all: $(TARGETS)
 
-xenpaging: $(OBJS)
+xenpaging: $(OBJS-y)
        $(CC) $(LDFLAGS) -o $@ $^ $(LDLIBS) $(APPEND_LDFLAGS)
 
 install: all
        $(INSTALL_DIR) -m 0700 $(DESTDIR)$(XEN_PAGING_DIR)
        $(INSTALL_DIR) $(DESTDIR)$(LIBEXEC_BIN)
-       $(INSTALL_PROG) $(IBINS) $(DESTDIR)$(LIBEXEC_BIN)
+       $(INSTALL_PROG) $(TARGETS) $(DESTDIR)$(LIBEXEC_BIN)
 
 uninstall:
-       rm -f $(addprefix $(DESTDIR)$(LIBEXEC_BIN)/, $(IBINS))
+       rm -f $(addprefix $(DESTDIR)$(LIBEXEC_BIN)/, $(TARGETS))
 
 clean:
-       rm -f *.o *~ $(DEPS_RM) xen TAGS $(IBINS) $(LIB)
+       rm -f *.o *~ $(DEPS_RM) TAGS $(TARGETS)
 
 distclean: clean
 
@@ -39,6 +39,6 @@ distclean: clean
 
 .PHONY: TAGS
 TAGS:
-       etags -t $(SRCS) *.h
+       etags -t *.c *.h
 
 -include $(DEPS_INCLUDE)