linux-kernel - Re: [PATCH 1/1] Makefile: Fix cross building

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CAK7LNAR1L=sh=TBj9ZjUZfckKPw4Ys=3UMaYfE+47VF5z=dyww@mail.gmail.com>
Date:   Wed, 21 Apr 2021 19:10:45 +0900
From:   Masahiro Yamada <masahiroy@...nel.org>
To:     Patrick Waterlander <patrick.waterlander@...el.com>
Cc:     nathan.d.ciobanu@...el.com, Michal Marek <michal.lkml@...kovi.net>,
        Linux Kbuild mailing list <linux-kbuild@...r.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 1/1] Makefile: Fix cross building

On Wed, Apr 21, 2021 at 9:20 AM Patrick Waterlander
<patrick.waterlander@...el.com> wrote:
>
> The native pkg-config is always called which may result in incorrect
> flags and paths being passed to the cross compiler. Implement
> $(CROSS_COMPILE)pkg-config to avoid this issue.
>
> Signed-off-by: Patrick Waterlander <patrick.waterlander@...el.com>
> ---
>  Makefile | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/Makefile b/Makefile
> index bc19584fee59..9aa2073f68d6 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -462,6 +462,7 @@ LZMA                = lzma
>  LZ4            = lz4c
>  XZ             = xz
>  ZSTD           = zstd
> +PKG_CONFIG     ?= $(CROSS_COMPILE)pkg-config
>
>  CHECKFLAGS     := -D__linux__ -Dlinux -D__STDC__ -Dunix -D__unix__ \
>                   -Wbitwise -Wno-return-void -Wno-unknown-attribute $(CF)
> @@ -1089,7 +1090,7 @@ mod_sign_cmd = true
>  endif
>  export mod_sign_cmd
>
> -HOST_LIBELF_LIBS = $(shell pkg-config libelf --libs 2>/dev/null || echo -lelf)
> +HOST_LIBELF_LIBS = $(shell $(PKG_CONFIG) libelf --libs 2>/dev/null || echo -lelf)


libelf will be linked to host programs.

This is clear from HOST_LIBELF_LIBS.
                   ^^^^

pkg-config should be OK.




>
>  has_libelf = $(call try-run,\
>                 echo "int main() {}" | $(HOSTCC) -xc -o /dev/null $(HOST_LIBELF_LIBS) -,1,0)
> --
> 2.17.1
>


-- 
Best Regards
Masahiro Yamada