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

Re: [XEN PATCH v3 11/25] tools/xentrace: rework Makefile


  • To: Luca Fancellu <Luca.Fancellu@xxxxxxx>
  • From: Anthony PERARD <anthony.perard@xxxxxxxxxx>
  • Date: Mon, 8 Aug 2022 16:35:13 +0100
  • Authentication-results: esa2.hc3370-68.iphmx.com; dkim=none (message not signed) header.i=none
  • Cc: "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>, "George Dunlap" <george.dunlap@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>
  • Delivery-date: Mon, 08 Aug 2022 15:35:32 +0000
  • Ironport-data: A9a23:h31NN6BJ+URFNRVWkefolKxfNB/EJm4LizeM4r3joE0InEoKPFUCq0HgJSnKGYA4Qc41jMxiXanNTTvWBpoYkwc3S4cn6jLgLYjshj2TamjNzvQeJpR1L3L6ckNe8Pv/S33kuYoXGJ4N4voonDMRdAbur0H3oZ2sgae/s5wy76/7HAlA9Cs/FZguI+QR7LXYCbK1ah7m5AjeH2wG2+upqEc0XNEkApJDxGP2crIBfbHjuYxE19XzrC5ZRgoUOj9QLMgn4I8np/gKZJyhtteHE+oWaEFCVDofjpckYfQ4Qzzzue+hhvZ7gi6fzW0Gx98rZnrVqz6vv38SNjT7/PM2OA3Va0gFq/VXHLVNy1SjD+SBo2JzePssHzTfXRWpIdVRD8gYiaj2o2xgnNrbuiDGM/TUQ8hQr4QOMAZSwLKlAQ9SqF9P1e1r9W1xQ48JOYwvV5mayHRk6AwGxHGDxU0J+96MbMCdxteN+UdDwhTiW8q5lwJA4QIg/1fJdO9VMaMedQYjnToJyGrpe++dNm9MOeVIjPxLGM96ne9oDv6A3fha7nQ9GpSfsRzVDLgGJ5cS9v2/4uofiXIuyxId44Osard6t5ZLoH2fDEcIEK9vpo6LO693/kjo+HmpmJj7hjVk/JgsWPdn8FsC5Cj4+7DGorNCOh6qS/7KP6wC9141+oSR7kyfQ59Bfr8oEu+trtQsX+bmIZlmK2wnt6cR+sODXMIouxWA0NGQWAHBSUy2IQx3MvznlbMQlNhN20dVSl2U+Z7pjswTGVwd2r/JjkMSu9ZRYjsLEkn27DS+34c5umZve2N67gZw9nzCgpauWqeli/+kLUa9WysI8bBBDFbvqADTgYvfYbyNbol7wPT+j0FymphIHmn9iYAb0oou42SHMy3FX/aFlJvkjBeVECtdQ0X+HqZ8qd/8gDh27i59hoNkK0P8+qQhMnSh8GxpKO3p3QlKZk2hA87d+P5DSrlul18Ybz6OPIRsJE2zPCBc9zdTj8nht32r6ti0cIQwx5Ku2DLsCStg9flvM84Pf4BYHSEf+g946xTCx9LzaIQQo2fVH5tCzh9BI4j2fNcm3BqiP4GpFGoAsTbSZvS/swDnEeS4rt1I7H26NxZ/D9oZO/UJCZo5psza9KJd7ZoOiHwTsHa710ZJ3nLzXuabx/CK1monl/8iW1M2yZrEf5bvYhP1/YBz0T4M6NaRAHDoJY4+mJSTCcCWBBqbPB52r47aJDSf2jqM5lEo1GTC/8MdNP5ki4Ao6oRkOoAAPIxgYv+UA9FCSI0JJgbbcV5bfjQ0sfTXym4sbeHkLNKtBYuuL5BTB0SI3bUAae3U7JCqEiokZTr4rXQen+A5w3+1rnZnVHkSH8N+EqMiBxKU1Uhtn6sSDjtQc6fWVqFyyNJaByNmb2BXDcy7KAXZ4xDO+jDoSy+X2dR3t3P5iVMhC5UKKIhz3CIcd9MACsQtbtwchry2GjhFIkFcB0Tq7WYoK+6kwWlZO2/uy/IHkL101iiNRzpQmUsWBuSxPKyndUycNqQnJq4ZbcFhw1CpeJfpHD+7PsMSq3h4UKlsvGbpOBoX3TfMFm9Jw7hf2ToN2eVdx/K202+Noch/SWFcWmP83SH/0HAJF8fez3biO2P80OwFpT6pA9tIYlMVU5UhobyzPhc3UnKZzo7CZ+KpQPMb7V4knGTWTtuplmtnwOnWBTyQV/NlTx+buCAtbOscnCAXw1qmr8wBkMf2JZEXL0JQscsyHJoK/X2/K0/nqeOI7BElF8+Uq8UaSbHZy73ef8dAMqZYGVfQYpJFh+qfmmA77eqGSZZOibtxkTdOkhqZ0CuiFhu9kVUsuI684BHiOwvO134dO/T2Z3dNyMlHeDAu1R2LQD8gNQ0GdAtFwyBrbiH1ipULkrrJzHo5vsUt6MHAcqsHc8kxsZm5zOlSiIpYRBCb43zNgX2uFvaQTvm9yZhGZlgOazeEc72SrszetiXu8ddLBrKvwm4axJGShJzTWNMAlDLZM4b7Aahhk1lyBjupJbpxdbIf4RtOyGi+CN+Ri+jiVh4grTJwGOwITQLTJXye2DxlTrqFLf6Obt2q2t/ZCnw8hfZO9wi4kK2+xhnS4r/OliA5iH6zcvItaM7JDIEQ1ZIFu6bAELE2bUnrG/O3lplo1c5Syw5Vp1Pe8gKX1vQfH1u3apTLmw8hHSDu+mGDIulkUvux4A22/DAxC+/2k9MykmsAkf9fe2bMB+HQAkztg9T1JUMdTIksJHgmlbMVW+EXqiGZrf7FDZ22bq1mappa21rg2Ua7Uw0X1tOKC3+vxwvTQn1/4MrcF4kkXrZ9ixYlEQKx3LAWsjWYn9VbPoMYtP2dOe1tJz4zXJp0y97hSL/X+QYpjyw+uSYlX+8syTQl188Lk8JVDhZA1QamnygOwt9KDBGlQQJNNJaNoXYXsd5lp7fDFZPs3lEOjm9DrWk0C9qwEV6NDuGA4oWy4w8Q6KC+nymzZ8HVXTc5Wls/EHlopkTp3B000WaC41nMvfh7k5qc9lj3HiQQdHOc0DatR0zjsVUHpEiWwVvBRPgvmoIfpN6mSSOn1Lr7FHEmtqMk96o+mA5s3JzWtnRXptqCwIZi7WKKGg3xRexq6gao5Fv0pS5cPVREgjqQ0QK92rIdbfGHHAh0d1ZGvLy2cb266tNQjb2nWSZWUeocITFZJxZEjNUhi6dq9Q==
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On Fri, Jul 22, 2022 at 01:30:53PM +0000, Luca Fancellu wrote:
> > On 24 Jun 2022, at 17:04, Anthony PERARD <anthony.perard@xxxxxxxxxx> 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



 


Rackspace

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