[Buildroot] [PATCH] package/micropython: Define MICROPY_NLR_SETJMP for xtensa

[Chris Packham]

On Mon, Apr 5, 2021 at 1:40 AM Thomas Petazzoni
<thomas.petazzoni at bootlin.com> wrote:
>
> On Mon, 29 Mar 2021 20:28:28 +1300
> Chris Packham <judge.packham at gmail.com> wrote:
>
> > As suggested on https://github.com/micropython/micropython/issues/6551
> > define MICROPY_NLR_SETJMP to avoid the xtensa specific implementation
> > of nlr_push.
> >
> > Fixes:
> > - http://autobuild.buildroot.net/results/5fc8669b5c768ccfc02bd20d1159bce7fe43683e
> >
> > Signed-off-by: Chris Packham <judge.packham at gmail.com>
> > ---
> >  package/micropython/micropython.mk | 5 +++++
> >  1 file changed, 5 insertions(+)
>
> The build error really looks like a toolchain issue, but fair enough as
> a temporary measure what you proposed is fine, so I've applied.
>
> Could you perhaps e-mail the issue to Max Filippov
> <jcmvbkbc at gmail.com> ? He has been maintaining the Xtensa support in BR,
> including toolchain fixes, for quite some time.

I believe Max is already on the issue I raised upstream[1].

I don't really know anything about xtensa but I suspect there might be
a difference between building for bare-metal vs building for an OS.
Either that or the xtensa specific code is just broken and no-one else
has noticed.

[1] - https://github.com/micropython/micropython/issues/6551

>
> Thanks a lot!
>
> Thomas
> --
> Thomas Petazzoni, CTO, Bootlin
> Embedded Linux and Kernel engineering
> https://bootlin.com