> On 19 Oct 2020, at 08:21, Jan Beulich <jbeul...@suse.com> wrote:
>
> This again was working right only as long as $(LIBHEADER) consisted of
> just one entry.
>
> Signed-off-by: Jan Beulich <jbeul...@suse.com>
Reviewed-by: Bertrand Marquis <bertrand.marq...@arm.com>
The change is obviously fixing a bug :-) and the double $ is required to
protect from make.
Cheers
Bertrand
> ---
> An alternative would be to use $(addprefix ) without any shell loop.
>
> --- a/tools/libs/libs.mk
> +++ b/tools/libs/libs.mk
> @@ -107,7 +107,7 @@ install: build
> .PHONY: uninstall
> uninstall:
> rm -f $(DESTDIR)$(PKG_INSTALLDIR)/$(LIB_FILE_NAME).pc
> - for i in $(LIBHEADER); do rm -f $(DESTDIR)$(includedir)/$(LIBHEADER);
> done
> + for i in $(LIBHEADER); do rm -f $(DESTDIR)$(includedir)/$$i; done
> rm -f $(DESTDIR)$(libdir)/lib$(LIB_FILE_NAME).so
> rm -f $(DESTDIR)$(libdir)/lib$(LIB_FILE_NAME).so.$(MAJOR)
> rm -f $(DESTDIR)$(libdir)/lib$(LIB_FILE_NAME).so.$(MAJOR).$(MINOR)
>
>
