On Fri, Jul 22, 2022 at 01:30:53PM +0000, Luca Fancellu wrote: > > On 24 Jun 2022, at 17:04, Anthony PERARD <anthony.per...@citrix.com> wrote: > > .PHONY: uninstall > > uninstall: > > rm -f $(addprefix $(DESTDIR)$(LIBEXEC_BIN)/, $(LIBBIN)) > > rm -f $(addprefix $(DESTDIR)$(bindir)/, $(SCRIPTS)) > > rm -f $(addprefix $(DESTDIR)$(sbindir)/, $(SBIN)) > > -ifneq ($(BIN),) > > rm -f $(addprefix $(DESTDIR)$(bindir)/, $(BIN)) > > -endif > > Why here don’t we use $(RM) ?
Well, I don't think it matters which is used between $(RM) and rm -f, beside consistency maybe. So I don't think introducing changes on those line would be useful. (Also, it seems that the use of $(RM) for "uninstall" targets are exceptional so far.) -- Anthony PERARD
