Page 1 of 1

Error building esp8266 port on pfalcon's branch

Posted: Sat Dec 01, 2018 1:54 pm
by mattzz
Hi,

I'm trying to build ports/esp8266 on pfalcon's branch/fork of micropython
I am on 8f429c8426ff2ea4d1d080ca24d179dedaebc1f2

Compiling master branch works fine.
Compiling pfalcon branch results in an error:

Code: Select all

...
MPY modules/picoweb/__init__.py
MPY modules/picoweb/utils.py
GEN build/frozen_mpy.c
CC build/frozen_mpy.c
CC strtoll.c
CC main.c
CC help.c
CC esp_mphal.c
CC esp_init_data.c
CC gccollect.c
gccollect.c:36:1: error: unknown type name 'mp_uint_t'
 mp_uint_t gc_helper_get_regs_and_sp(mp_uint_t *regs);
 ^
gccollect.c:36:37: error: unknown type name 'mp_uint_t'
 mp_uint_t gc_helper_get_regs_and_sp(mp_uint_t *regs);
                                     ^
gccollect.c: In function 'gc_collect':
gccollect.c:43:5: error: unknown type name 'mp_uint_t'
     mp_uint_t regs[8];
     ^
gccollect.c:44:5: error: unknown type name 'mp_uint_t'
     mp_uint_t sp = gc_helper_get_regs_and_sp(regs);
     ^
gccollect.c:44:5: error: implicit declaration of function 'gc_helper_get_regs_and_sp' [-Werror=implicit-function-declaration]
cc1: all warnings being treated as errors
make: *** [build/gccollect.o] Error 1
gc_helper_get_regs_and_sp is defined in gchelper.s, however, in the verbose compile output (make V=1) I can't find any reference about gchelper.o.

Am I missing something?

Thanks,
/Matthias

Re: Error building esp8266 port on pfalcon's branch

Posted: Sat Dec 01, 2018 2:48 pm
by jomas
gccollect.c:36:1: error: unknown type name 'mp_uint_t'
is the problem. mp_uint_t is not defined.
You can fix this by including mpconfigport.h in the beginning of file: gccollect.c

Re: Error building esp8266 port on pfalcon's branch

Posted: Sat Dec 01, 2018 5:05 pm
by pythoncoder
Raise an issue against that fork?

Re: Error building esp8266 port on pfalcon's branch

Posted: Sat Dec 01, 2018 8:40 pm
by pfalcon
Thanks for the report. Turns out, CI wasn't run on esp8266 port for a while, both in mainline in my fork. Fixed and confirmed via CI: https://travis-ci.org/pfalcon/micropyth ... /462270955

Note that my branch is rebased. To pull new changes, you always need to use "git pull --rebase".