[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