Messages in this thread | | | Subject | Re: [PATCH v3 4/4] kconfig: refactor ncurses package checks for building mconf and nconf | From | Randy Dunlap <> | Date | Tue, 22 May 2018 08:36:11 -0700 |
| |
On 05/22/2018 12:22 AM, Masahiro Yamada wrote: > The mconf (or its infrastructure, lxdiaglog) depends on the ncurses. > Move and rename check-lxdialog.sh to mconf-cfg.sh to make it work in > the same way as for qconf and gconf. > > This commit fixes some more weirdnesses. > > The nconf also needs ncurses packages. HOSTLOADLIBES_nconf is set > to the libraries needed for nconf, but the cflags is not explicitly > set. Actually, nconf relies on the check-lxdialog.sh for the proper > cflags: > > HOST_EXTRACFLAGS += $(shell $(CONFIG_SHELL) $(check-lxdialog) -ccflags) \ > -DLOCALE > > The code above passes the ncurses flags to all objects, even for conf, > qconf, gconf. Let's pass the ncurses flags only to mconf and nconf. > > Currently, the presence of ncurses is not checked for nconf. Let's > show a prompt like the mconf case. > > According to Randy's report, the shell scripts still need to carry > the fallback code in case the pkg-config fails to find the ncurses > packages. > > Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
Tested-by: Randy Dunlap <rdunlap@infradead.org> Acked-by: Randy Dunlap <rdunlap@infradead.org>
Thanks.
> --- > > Changes in v3: > - Squash two patches into one to not lose bisectability > > Changes in v2: > - Restore fallback detection for openSUSE etc. > > scripts/kconfig/Makefile | 58 ++++++++----------- > scripts/kconfig/lxdialog/check-lxdialog.sh | 93 ------------------------------ > scripts/kconfig/lxdialog/dialog.h | 2 +- > scripts/kconfig/mconf-cfg.sh | 44 ++++++++++++++ > scripts/kconfig/nconf-cfg.sh | 43 ++++++++++++++ > 5 files changed, 112 insertions(+), 128 deletions(-) > delete mode 100755 scripts/kconfig/lxdialog/check-lxdialog.sh > create mode 100755 scripts/kconfig/mconf-cfg.sh > create mode 100644 scripts/kconfig/nconf-cfg.sh
-- ~Randy
| |