Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Jason Wessel <jason.wessel@windriver.com>
To: <Openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH] ncurses, busybox, cml1.bbclass: Fix menuconfig display corruption
Date: Mon, 4 Mar 2013 15:56:47 -0600	[thread overview]
Message-ID: <5135189F.5070203@windriver.com> (raw)
In-Reply-To: <1362432115-43230-1-git-send-email-jason.wessel@windriver.com>

On 03/04/2013 03:21 PM, Jason Wessel wrote:
> Previously there was a change to the ncurses compile to make it more
> like the typical way it was compiled on a host system.  This fixed a
> whole class of host machines, but masked the real underlying problem
> with the display corruption issues and menuconfig.
>
> The corner case that led to the discovery that the wrong curses.h file
> was getting used was when there was no curses libraries at all on one
> of the development hosts.  What had happened before was that
> /usr/include/curses.h on the host system had to match closely enough
> to the curses.h in the sysroot and then linking against the sysroot
> version of curses.so was ok (meaning no display corruption).  But on
> some systems with ncurses.h vs curses.h such as SuSE hosts, there were
> still issues.
>
> If we fix the root of the problem and force the mconf and lxdialog to
> use the correct headers and libraries from the sysroot there is no
> further issues and the menuconfig target works properly.  It also
> means we can back out the custom compilation flags to the ncurses
> recipe because they are no longer needed.
>
> For the kernel part of the menuconfig / nconfig changes it will be
> merged separately and this is all based on:
>
> https://lkml.org/lkml/2013/3/3/103

Probably best to hold off merging this until we see if the kernel patch is accepted upstream.  This is more of an RFC and to raise further awareness of the problem and the fact that we have a solution now that the root cause is well understood on the menuconfig corruption.

Cheers,
Jason.



  reply	other threads:[~2013-03-04 22:13 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-03-04 21:21 [PATCH] ncurses, busybox, cml1.bbclass: Fix menuconfig display corruption Jason Wessel
2013-03-04 21:56 ` Jason Wessel [this message]
2013-03-04 22:11   ` Trevor Woerner
2013-03-04 22:14     ` Jason Wessel

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5135189F.5070203@windriver.com \
    --to=jason.wessel@windriver.com \
    --cc=Openembedded-core@lists.openembedded.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox