[CRIU] [PATCH 1/2] Makefile nitpicks
Kir Kolyshkin
kir at openvz.org
Mon Sep 19 14:01:28 PDT 2016
On 09/19/2016 01:58 PM, Kir Kolyshkin wrote:
> 1. As __nmk_dir already ends with a slash, there's no need
> for an extra one.
>
> 2. No need to include macro.mk into criu/Makefile -- nothing it defines
> is used from there.
>
> Cc: Cyrill Gorcunov <gorcunov at openvz.org>
> Signed-off-by: Kir Kolyshkin <kir at openvz.org>
> ---
> Documentation/Makefile | 4 ++--
> Makefile | 4 ++--
> criu/Makefile | 2 +-
> criu/Makefile.config | 2 +-
> lib/Makefile | 3 +--
> 5 files changed, 7 insertions(+), 8 deletions(-)
>
> diff --git a/Documentation/Makefile b/Documentation/Makefile
> index 77c4281..b1d7c79 100644
> --- a/Documentation/Makefile
> +++ b/Documentation/Makefile
> @@ -1,6 +1,6 @@
> __nmk_dir ?= ../scripts/nmk/scripts/
> -include $(__nmk_dir)/include.mk
> -include $(__nmk_dir)/macro.mk
> +include $(__nmk_dir)include.mk
> +include $(__nmk_dir)macro.mk
Ughm, this needs to be applied *after* a patch from the other series
named "[PATCH 1/4] Documentation/Makefile: set nmkdir". Sorry for the mess.
>
> ASCIIDOC := asciidoc
> A2X := a2x
> diff --git a/Makefile b/Makefile
> index 05946a1..2536f7a 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -3,8 +3,8 @@
> __nmk_dir=$(CURDIR)/scripts/nmk/scripts/
> export __nmk_dir
>
> -include $(__nmk_dir)/include.mk
> -include $(__nmk_dir)/macro.mk
> +include $(__nmk_dir)include.mk
> +include $(__nmk_dir)macro.mk
>
> #
> # To build host helpers.
> diff --git a/criu/Makefile b/criu/Makefile
> index ba973e7..79cf23b 100644
> --- a/criu/Makefile
> +++ b/criu/Makefile
> @@ -35,7 +35,7 @@ ifneq ($(filter ia32 x86 ppc64,$(ARCH)),)
> endif
>
> # msg-* printing
> -include $(__nmk_dir)/msg.mk
> +include $(__nmk_dir)msg.mk
>
> #
> # Needed libraries checks
> diff --git a/criu/Makefile.config b/criu/Makefile.config
> index 73ba005..f79868b 100644
> --- a/criu/Makefile.config
> +++ b/criu/Makefile.config
> @@ -1,4 +1,4 @@
> -include $(__nmk_dir)/utils.mk
> +include $(__nmk_dir)utils.mk
> include $(__nmk_dir)msg.mk
> include $(SRC_DIR)/scripts/feature-tests.mak
>
> diff --git a/lib/Makefile b/lib/Makefile
> index f1c0821..d036893 100644
> --- a/lib/Makefile
> +++ b/lib/Makefile
> @@ -1,5 +1,4 @@
> -include $(__nmk_dir)/include.mk
> -include $(__nmk_dir)/macro.mk
> +include $(__nmk_dir)include.mk
>
> CRIU_SO := libcriu.so
> UAPI_HEADERS := c/criu.h ../images/rpc.proto
More information about the CRIU
mailing list