[CRIU] [PATCH] Makefile: fix libcriu.so links and man page installation

Christopher Covington cov at codeaurora.org
Fri Feb 7 07:24:56 PST 2014


On 02/07/2014 10:18 AM, Adrian Reber wrote:
> 
> The links to the versioned libcriu.so.* included $(DESTDIR) and
> $(LIBDIR) which will generate wrong links if building criu packages.
> Now the links points to the correct file without any path.
> 
> For some reason criu.8 was installed with executable permission. Use
> 'install' with '-m 644' to make sure this does not happen.

I would suggest symbolic notation such as a=r for easier comprehension.

Christopher

> Signed-off-by: Adrian Reber <adrian at lisas.de>
> ---
>  Documentation/Makefile | 2 +-
>  Makefile               | 4 ++--
>  2 files changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/Documentation/Makefile b/Documentation/Makefile
> index 50e4526..e236635 100644
> --- a/Documentation/Makefile
> +++ b/Documentation/Makefile
> @@ -46,6 +46,6 @@ clean:
>  install: $(MANS)
>  	$(E) "  INSTALL " $(MANS)
>  	$(Q) mkdir -p $(DESTDIR)$(MAN8DIR)
> -	$(Q) install $(MANS) $(DESTDIR)$(MAN8DIR)
> +	$(Q) install -m 644 $(MANS) $(DESTDIR)$(MAN8DIR)
>  
>  .PHONY: clean install
> diff --git a/Makefile b/Makefile
> index 9ed45bd..a4d560a 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -241,9 +241,9 @@ install: $(PROGRAM) install-man
>  	$(Q) mkdir -p $(DESTDIR)$(LIBDIR)
>  	$(Q) install -m 755 $(CRIU-LIB) \
>  		$(DESTDIR)$(LIBDIR)/$(CRIU-SO).so.$(VERSION_SO_MAJOR).$(VERSION_SO_MINOR)
> -	$(Q) ln -s $(DESTDIR)$(LIBDIR)/$(CRIU-SO).so.$(VERSION_SO_MAJOR).$(VERSION_SO_MINOR) \
> +	$(Q) ln -s $(CRIU-SO).so.$(VERSION_SO_MAJOR).$(VERSION_SO_MINOR) \
>  		$(DESTDIR)$(LIBDIR)/$(CRIU-SO).so.$(VERSION_SO_MAJOR)
> -	$(Q) ln -s $(DESTDIR)$(LIBDIR)/$(CRIU-SO).so.$(VERSION_SO_MAJOR).$(VERSION_SO_MINOR) \
> +	$(Q) ln -s $(CRIU-SO).so.$(VERSION_SO_MAJOR).$(VERSION_SO_MINOR) \
>  		$(DESTDIR)$(LIBDIR)/$(CRIU-SO).so
>  	$(Q) mkdir -p $(DESTDIR)$(INCLUDEDIR)
>  	$(Q) install -m 644 $(CRIU-INC) $(DESTDIR)$(INCLUDEDIR)
> 


-- 
Employee of Qualcomm Innovation Center, Inc.
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
hosted by the Linux Foundation.


More information about the CRIU mailing list