linux-kernel - Re: [PATCH] menuconfig,mconf-cfg: Allow specification of ncurses location

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <a5ea40a5-17ef-f037-16b3-177f01db6447@broadcom.com>
Date:   Tue, 1 Dec 2020 09:19:34 -0800
From:   Scott Branden <scott.branden@...adcom.com>
To:     Masahiro Yamada <masahiroy@...nel.org>
Cc:     Linux Kbuild mailing list <linux-kbuild@...r.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        Broadcom Kernel Feedback List 
        <bcm-kernel-feedback-list@...adcom.com>,
        Bruce Ashfield <bruce.ashfield@...il.com>,
        OE-core <openembedded-core@...ts.openembedded.org>,
        openembeded-devel <openembedded-devel@...ts.openembedded.org>,
        Michal Marek <mmarek@...e.cz>
Subject: Re: [PATCH] menuconfig,mconf-cfg: Allow specification of ncurses
 location

Hi Masahiro,

On 2020-12-01 4:25 a.m., Masahiro Yamada wrote:
> On Sat, Nov 28, 2020 at 9:45 AM Scott Branden
> <scott.branden@...adcom.com> wrote:
>> From: Bruce Ashfield <bruce.ashfield@...driver.com>
>>
>> In some cross build environments such as the Yocto Project build
>> environment it provides an ncurses library that is compiled
>> differently than the host's version.  This causes display corruption
>> problems when the host's curses includes are used instead of the
>> includes from the provided compiler are overridden.  There is a second
>> case where there is no curses libraries at all on the host system and
>> menuconfig will just fail entirely.
>>
>> The solution is simply to allow an override variable in
>> check-lxdialog.sh for environments such as the Yocto Project.  Adding
>> a CROSS_CURSES_LIB and CROSS_CURSES_INC solves the issue and allowing
>> compiling and linking against the right headers and libraries.
>>
>> Signed-off-by: Jason Wessel <jason.wessel@...driver.com>
>> cc: Michal Marek <mmarek@...e.cz>
>> cc: linux-kbuild@...r.kernel.org
>> Signed-off-by: Bruce Ashfield <bruce.ashfield@...driver.com>
>> Signed-off-by: Scott Branden <scott.branden@...adcom.com>
>> ---
>
> Some people solve the cross-compiling in Yocto
> by using pkg-config.
>
>
> For example,
>
> commit 067c650c456e758f933aaf87a202f841d34be269
> Author: Pavel Modilaynen <pavel.modilaynen@...s.com>
> Date:   Fri Jul 12 13:52:19 2019 +0200
>
>     dtc: Use pkg-config to locate libyaml
>
>     Using Makefile's wildcard with absolute path to detect
>     the presence of libyaml results in false-positive
>     detection when cross-compiling e.g. in yocto environment.
>
>
>
> mconf-cfg.sh already allows the path flexibility with pkg-config.
> Why do you want yet another hook?
I hope the yocto community can provide more details on this patch.
The yocto environment isolates the build environment from the host tools.
Running menuconfig with the upstream kernel does not work on the latest yocto without this patch.
>>  scripts/kconfig/mconf-cfg.sh | 8 ++++++++
>>  1 file changed, 8 insertions(+)
>>  mode change 100755 => 100644 scripts/kconfig/mconf-cfg.sh
>>
>> diff --git a/scripts/kconfig/mconf-cfg.sh b/scripts/kconfig/mconf-cfg.sh
>> old mode 100755
>> new mode 100644
>> index aa68ec95620d..32448bc198a5
>> --- a/scripts/kconfig/mconf-cfg.sh
>> +++ b/scripts/kconfig/mconf-cfg.sh
>> @@ -4,6 +4,14 @@
>>  PKG="ncursesw"
>>  PKG2="ncurses"
>>
>> +if [ "$CROSS_CURSES_LIB" != "" ]; then
>> +    echo libs=\'$CROSS_CURSES_LIB\'
>> +    if [ x"$CROSS_CURSES_INC" != x ]; then
>> +       echo cflags=\'$CROSS_CURSES_INC\'
>> +    fi
>> +    exit 0
>> +fi
>> +
>>  if [ -n "$(command -v pkg-config)" ]; then
>>         if pkg-config --exists $PKG; then
>>                 echo cflags=\"$(pkg-config --cflags $PKG)\"
>> --
>> 2.17.1
>>
>


Download attachment "smime.p7s" of type "application/pkcs7-signature" (4169 bytes)