[OE-core] [PATCH] terminal: Disable shopt errexit
Richard Tollerton
rich.tollerton at ni.com
Tue Mar 31 21:32:54 UTC 2015
"Burton, Ross" <ross.burton at intel.com> writes:
> On 30 March 2015 at 19:11, Richard Tollerton <rich.tollerton at ni.com> wrote:
>
>> Running `false` interactively doesn't test this behavior because `sh -e`
>> explicitly excludes interactive shells from consideration [1] [2].
>> You'll need to inject a `false` into run.do_terminal in order to
>> reproduce this.
>>
>
> I'm clearly not understanding how this behaviour is a problem then. What
> broke?
What broke is devshell. If there's a bug in run.do_terminal that would
cause a nonzero exit code during non-interactive execution -- e.g.,
http://cgit.openembedded.org/bitbake/commit/?id=18cd0ce6a55c9065c3f1bf223b47d817b5efcd8f
-- then instead of a warning appearing in the devshell, the entire
terminal quits a few milliseconds after it loads.
In other words, this issue escalates other bugs that ought to be
warnings into functionality-killers. This patch de-escalates them
back to warnings.
> Ross
More information about the Openembedded-core
mailing list