Re: [linux-yocto] [PATCH] scripts/kconfig: Fix menuconfig failure
On 2018-09-11 1:55 AM, He Zhe wrote: On 2018年08月15日 00:21, Bruce Ashfield wrote: On 08/14/2018 03:59 AM, He Zhe wrote: On 2018年08月13日 23:38, Bruce Ashfield wrote: Which kernel version(s) are you seeing this issue ? Just linux-yocto-dev ? Yes, just linux-yocto-dev. The following porting for linux-yocto-dev changes a bit This is now merged. Seems this is not merged yet. It this not it ? commit 3c70e6fd7e5f290d55dc49f634579bf65128b167 Author: He Zhe Date: Sat Aug 11 08:11:02 2018 -0700 scripts/kconfig: Fix menuconfig failure The folloing porting lacks of change log and breaks menuconfig. CROSS_CURSES_LIB or CROSS_CURSES_INC contains xxx"" which is finially evaluated as "xxx""" in which < and > would be considered as I/O redirection and cause the faliure below. 4ead19c77a5baaf44bfa6600b1be884e47c9d8f1 "menuconfig,mconf-cfg: Allow specification of ncurses location" scripts/kconfig/.mconf-cfg: line 2: curses.h: No such file or directory ... mconf.c:(.text+0x894): undefined reference to `stdscr' ... scripts/Makefile.host:99: recipe for target 'scripts/kconfig/mconf' failed This patch turns to use single quotes. Signed-off-by: He Zhe Signed-off-by: Bruce Ashfield yow-bashfiel-d4 [/home/bruc...to-dev.git]> git branch --contains 3c70e6fd7e5f290d55dc49f634579bf65128b167 standard/arm-versatile-926ejs * standard/base standard/beaglebone standard/edgerouter standard/fsl-mpc8315e-rdb standard/intel standard/mti-malta32 standard/mti-malta64 standard/preempt-rt/base standard/preempt-rt/intel standard/qemuarm64 standard/qemuppc standard/tiny/arm-versatile-926ejs standard/tiny/base standard/tiny/common-pc standard/tiny/intel Bruce Zhe Bruce from linux-yocto: --- a/scripts/kconfig/lxdialog/check-lxdialog.sh +++ b/scripts/kconfig/lxdialog/check-lxdialog.sh @@ -4,6 +4,10 @@ # What library to link ldflags() { + if [ "$CROSS_CURSES_LIB" != "" ]; then + echo "$CROSS_CURSES_LIB" + exit + fi pkg-config --libs ncursesw 2>/dev/null && exit pkg-config --libs ncurses 2>/dev/null && exit for ext in so a dll.a dylib ; do @@ -21,6 +25,10 @@ ldflags() # Where is ncurses.h? ccflags() { + if [ x"$CROSS_CURSES_INC" != x ]; then + echo "$CROSS_CURSES_INC" + exit + fi to linux-yocto-dev: diff --git a/scripts/kconfig/mconf-cfg.sh b/scripts/kconfig/mconf-cfg.sh index e6f9fac..557be6d 100755 --- 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 + Zhe Bruce On 08/11/2018 11:11 AM, He Zhe wrote: The folloing porting lacks of change log and breaks menuconfig. CROSS_CURSES_LIB or CROSS_CURSES_INC contains xxx"" which is finially evaluated as "xxx""" in which < and > would be considered as I/O redirection and cause the faliure below. 4ead19c77a5baaf44bfa6600b1be884e47c9d8f1 "menuconfig,mconf-cfg: Allow specification of ncurses location" scripts/kconfig/.mconf-cfg: line 2: curses.h: No such file or directory ... mconf.c:(.text+0x894): undefined reference to `stdscr' ... scripts/Makefile.host:99: recipe for target 'scripts/kconfig/mconf' failed This patch turns to use single quotes. Signed-off-by: He Zhe --- scripts/kconfig/mconf-cfg.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/kconfig/mconf-cfg.sh b/scripts/kconfig/mconf-cfg.sh index 557be6dc1abf..3bd1f9a363e6 100755 --- a/scripts/kconfig/mconf-cfg.sh +++ b/scripts/kconfig/mconf-cfg.sh @@ -5,9 +5,9 @@ PKG="ncursesw" PKG2="ncurses" if [ "$CROSS_CURSES_LIB" != "" ]; then - echo libs=\"$CROSS_CURSES_LIB\" + echo libs=\'$CROSS_CURSES_LIB\' if [ x"$CROSS_CURSES_INC" != x ]; then - echo cflags=\"$CROSS_CURSES_INC\" + echo cflags=\'$CROSS_CURSES_INC\' fi exit 0 fi -- ___ linux-yocto mailing list linux-yocto@yoctoproject.org https://lists.yoctoproject.org/listinfo/linux-yocto
Re: [linux-yocto] [PATCH] scripts/kconfig: Fix menuconfig failure
On 2018年08月15日 00:21, Bruce Ashfield wrote: > On 08/14/2018 03:59 AM, He Zhe wrote: >> >> >> On 2018年08月13日 23:38, Bruce Ashfield wrote: >>> Which kernel version(s) are you seeing this issue ? Just linux-yocto-dev ? >> >> Yes, just linux-yocto-dev. The following porting for linux-yocto-dev changes >> a bit > > This is now merged. Seems this is not merged yet. Zhe > > Bruce > >> >> from linux-yocto: >> >> --- a/scripts/kconfig/lxdialog/check-lxdialog.sh >> +++ b/scripts/kconfig/lxdialog/check-lxdialog.sh >> @@ -4,6 +4,10 @@ >> # What library to link >> ldflags() >> { >> + if [ "$CROSS_CURSES_LIB" != "" ]; then >> + echo "$CROSS_CURSES_LIB" >> + exit >> + fi >> pkg-config --libs ncursesw 2>/dev/null && exit >> pkg-config --libs ncurses 2>/dev/null && exit >> for ext in so a dll.a dylib ; do >> @@ -21,6 +25,10 @@ ldflags() >> # Where is ncurses.h? >> ccflags() >> { >> + if [ x"$CROSS_CURSES_INC" != x ]; then >> + echo "$CROSS_CURSES_INC" >> + exit >> + fi >> >> to linux-yocto-dev: >> >> diff --git a/scripts/kconfig/mconf-cfg.sh b/scripts/kconfig/mconf-cfg.sh >> index e6f9fac..557be6d 100755 >> --- 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 >> + >> >> >> Zhe >> >> >> >>> >>> Bruce >>> >>> On 08/11/2018 11:11 AM, He Zhe wrote: The folloing porting lacks of change log and breaks menuconfig. CROSS_CURSES_LIB or CROSS_CURSES_INC contains xxx"" which is finially evaluated as "xxx""" in which < and > would be considered as I/O redirection and cause the faliure below. 4ead19c77a5baaf44bfa6600b1be884e47c9d8f1 "menuconfig,mconf-cfg: Allow specification of ncurses location" scripts/kconfig/.mconf-cfg: line 2: curses.h: No such file or directory ... mconf.c:(.text+0x894): undefined reference to `stdscr' ... scripts/Makefile.host:99: recipe for target 'scripts/kconfig/mconf' failed This patch turns to use single quotes. Signed-off-by: He Zhe --- scripts/kconfig/mconf-cfg.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/kconfig/mconf-cfg.sh b/scripts/kconfig/mconf-cfg.sh index 557be6dc1abf..3bd1f9a363e6 100755 --- a/scripts/kconfig/mconf-cfg.sh +++ b/scripts/kconfig/mconf-cfg.sh @@ -5,9 +5,9 @@ PKG="ncursesw" PKG2="ncurses" if [ "$CROSS_CURSES_LIB" != "" ]; then - echo libs=\"$CROSS_CURSES_LIB\" + echo libs=\'$CROSS_CURSES_LIB\' if [ x"$CROSS_CURSES_INC" != x ]; then - echo cflags=\"$CROSS_CURSES_INC\" + echo cflags=\'$CROSS_CURSES_INC\' fi exit 0 fi >>> >>> >> > > -- ___ linux-yocto mailing list linux-yocto@yoctoproject.org https://lists.yoctoproject.org/listinfo/linux-yocto
Re: [linux-yocto] [PATCH] scripts/kconfig: Fix menuconfig failure
On 08/14/2018 03:59 AM, He Zhe wrote: On 2018年08月13日 23:38, Bruce Ashfield wrote: Which kernel version(s) are you seeing this issue ? Just linux-yocto-dev ? Yes, just linux-yocto-dev. The following porting for linux-yocto-dev changes a bit This is now merged. Bruce from linux-yocto: --- a/scripts/kconfig/lxdialog/check-lxdialog.sh +++ b/scripts/kconfig/lxdialog/check-lxdialog.sh @@ -4,6 +4,10 @@ # What library to link ldflags() { + if [ "$CROSS_CURSES_LIB" != "" ]; then + echo "$CROSS_CURSES_LIB" + exit + fi pkg-config --libs ncursesw 2>/dev/null && exit pkg-config --libs ncurses 2>/dev/null && exit for ext in so a dll.a dylib ; do @@ -21,6 +25,10 @@ ldflags() # Where is ncurses.h? ccflags() { + if [ x"$CROSS_CURSES_INC" != x ]; then + echo "$CROSS_CURSES_INC" + exit + fi to linux-yocto-dev: diff --git a/scripts/kconfig/mconf-cfg.sh b/scripts/kconfig/mconf-cfg.sh index e6f9fac..557be6d 100755 --- 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 + Zhe Bruce On 08/11/2018 11:11 AM, He Zhe wrote: The folloing porting lacks of change log and breaks menuconfig. CROSS_CURSES_LIB or CROSS_CURSES_INC contains xxx"" which is finially evaluated as "xxx""" in which < and > would be considered as I/O redirection and cause the faliure below. 4ead19c77a5baaf44bfa6600b1be884e47c9d8f1 "menuconfig,mconf-cfg: Allow specification of ncurses location" scripts/kconfig/.mconf-cfg: line 2: curses.h: No such file or directory ... mconf.c:(.text+0x894): undefined reference to `stdscr' ... scripts/Makefile.host:99: recipe for target 'scripts/kconfig/mconf' failed This patch turns to use single quotes. Signed-off-by: He Zhe --- scripts/kconfig/mconf-cfg.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/kconfig/mconf-cfg.sh b/scripts/kconfig/mconf-cfg.sh index 557be6dc1abf..3bd1f9a363e6 100755 --- a/scripts/kconfig/mconf-cfg.sh +++ b/scripts/kconfig/mconf-cfg.sh @@ -5,9 +5,9 @@ PKG="ncursesw" PKG2="ncurses" if [ "$CROSS_CURSES_LIB" != "" ]; then - echo libs=\"$CROSS_CURSES_LIB\" + echo libs=\'$CROSS_CURSES_LIB\' if [ x"$CROSS_CURSES_INC" != x ]; then - echo cflags=\"$CROSS_CURSES_INC\" + echo cflags=\'$CROSS_CURSES_INC\' fi exit 0 fi -- ___ linux-yocto mailing list linux-yocto@yoctoproject.org https://lists.yoctoproject.org/listinfo/linux-yocto
Re: [linux-yocto] [PATCH] scripts/kconfig: Fix menuconfig failure
On 2018年08月13日 23:38, Bruce Ashfield wrote: > Which kernel version(s) are you seeing this issue ? Just linux-yocto-dev ? Yes, just linux-yocto-dev. The following porting for linux-yocto-dev changes a bit from linux-yocto: --- a/scripts/kconfig/lxdialog/check-lxdialog.sh +++ b/scripts/kconfig/lxdialog/check-lxdialog.sh @@ -4,6 +4,10 @@ # What library to link ldflags() { + if [ "$CROSS_CURSES_LIB" != "" ]; then + echo "$CROSS_CURSES_LIB" + exit + fi pkg-config --libs ncursesw 2>/dev/null && exit pkg-config --libs ncurses 2>/dev/null && exit for ext in so a dll.a dylib ; do @@ -21,6 +25,10 @@ ldflags() # Where is ncurses.h? ccflags() { + if [ x"$CROSS_CURSES_INC" != x ]; then + echo "$CROSS_CURSES_INC" + exit + fi to linux-yocto-dev: diff --git a/scripts/kconfig/mconf-cfg.sh b/scripts/kconfig/mconf-cfg.sh index e6f9fac..557be6d 100755 --- 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 + Zhe > > Bruce > > On 08/11/2018 11:11 AM, He Zhe wrote: >> The folloing porting lacks of change log and breaks menuconfig. >> CROSS_CURSES_LIB or CROSS_CURSES_INC contains xxx"" which is >> finially evaluated as "xxx""" in which < and > would be >> considered as I/O redirection and cause the faliure below. >> >> 4ead19c77a5baaf44bfa6600b1be884e47c9d8f1 >> "menuconfig,mconf-cfg: Allow specification of ncurses location" >> >> scripts/kconfig/.mconf-cfg: line 2: curses.h: No such file or directory >> ... >> mconf.c:(.text+0x894): undefined reference to `stdscr' >> ... >> scripts/Makefile.host:99: recipe for target 'scripts/kconfig/mconf' failed >> >> This patch turns to use single quotes. >> >> Signed-off-by: He Zhe >> --- >> scripts/kconfig/mconf-cfg.sh | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/scripts/kconfig/mconf-cfg.sh b/scripts/kconfig/mconf-cfg.sh >> index 557be6dc1abf..3bd1f9a363e6 100755 >> --- a/scripts/kconfig/mconf-cfg.sh >> +++ b/scripts/kconfig/mconf-cfg.sh >> @@ -5,9 +5,9 @@ PKG="ncursesw" >> PKG2="ncurses" >> if [ "$CROSS_CURSES_LIB" != "" ]; then >> - echo libs=\"$CROSS_CURSES_LIB\" >> + echo libs=\'$CROSS_CURSES_LIB\' >> if [ x"$CROSS_CURSES_INC" != x ]; then >> - echo cflags=\"$CROSS_CURSES_INC\" >> + echo cflags=\'$CROSS_CURSES_INC\' >> fi >> exit 0 >> fi >> > > -- ___ linux-yocto mailing list linux-yocto@yoctoproject.org https://lists.yoctoproject.org/listinfo/linux-yocto
Re: [linux-yocto] [PATCH] scripts/kconfig: Fix menuconfig failure
Which kernel version(s) are you seeing this issue ? Just linux-yocto-dev ? Bruce On 08/11/2018 11:11 AM, He Zhe wrote: The folloing porting lacks of change log and breaks menuconfig. CROSS_CURSES_LIB or CROSS_CURSES_INC contains xxx"" which is finially evaluated as "xxx""" in which < and > would be considered as I/O redirection and cause the faliure below. 4ead19c77a5baaf44bfa6600b1be884e47c9d8f1 "menuconfig,mconf-cfg: Allow specification of ncurses location" scripts/kconfig/.mconf-cfg: line 2: curses.h: No such file or directory ... mconf.c:(.text+0x894): undefined reference to `stdscr' ... scripts/Makefile.host:99: recipe for target 'scripts/kconfig/mconf' failed This patch turns to use single quotes. Signed-off-by: He Zhe --- scripts/kconfig/mconf-cfg.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/kconfig/mconf-cfg.sh b/scripts/kconfig/mconf-cfg.sh index 557be6dc1abf..3bd1f9a363e6 100755 --- a/scripts/kconfig/mconf-cfg.sh +++ b/scripts/kconfig/mconf-cfg.sh @@ -5,9 +5,9 @@ PKG="ncursesw" PKG2="ncurses" if [ "$CROSS_CURSES_LIB" != "" ]; then - echo libs=\"$CROSS_CURSES_LIB\" + echo libs=\'$CROSS_CURSES_LIB\' if [ x"$CROSS_CURSES_INC" != x ]; then - echo cflags=\"$CROSS_CURSES_INC\" + echo cflags=\'$CROSS_CURSES_INC\' fi exit 0 fi -- ___ linux-yocto mailing list linux-yocto@yoctoproject.org https://lists.yoctoproject.org/listinfo/linux-yocto
[linux-yocto] [PATCH] scripts/kconfig: Fix menuconfig failure
The folloing porting lacks of change log and breaks menuconfig. CROSS_CURSES_LIB or CROSS_CURSES_INC contains xxx"" which is finially evaluated as "xxx""" in which < and > would be considered as I/O redirection and cause the faliure below. 4ead19c77a5baaf44bfa6600b1be884e47c9d8f1 "menuconfig,mconf-cfg: Allow specification of ncurses location" scripts/kconfig/.mconf-cfg: line 2: curses.h: No such file or directory ... mconf.c:(.text+0x894): undefined reference to `stdscr' ... scripts/Makefile.host:99: recipe for target 'scripts/kconfig/mconf' failed This patch turns to use single quotes. Signed-off-by: He Zhe --- scripts/kconfig/mconf-cfg.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/kconfig/mconf-cfg.sh b/scripts/kconfig/mconf-cfg.sh index 557be6dc1abf..3bd1f9a363e6 100755 --- a/scripts/kconfig/mconf-cfg.sh +++ b/scripts/kconfig/mconf-cfg.sh @@ -5,9 +5,9 @@ PKG="ncursesw" PKG2="ncurses" if [ "$CROSS_CURSES_LIB" != "" ]; then - echo libs=\"$CROSS_CURSES_LIB\" + echo libs=\'$CROSS_CURSES_LIB\' if [ x"$CROSS_CURSES_INC" != x ]; then - echo cflags=\"$CROSS_CURSES_INC\" + echo cflags=\'$CROSS_CURSES_INC\' fi exit 0 fi -- 2.11.0 -- ___ linux-yocto mailing list linux-yocto@yoctoproject.org https://lists.yoctoproject.org/listinfo/linux-yocto