Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unknown type name 'ppp_pcb' #1

Closed
jfdona23 opened this issue Jun 4, 2019 · 2 comments
Closed

Unknown type name 'ppp_pcb' #1

jfdona23 opened this issue Jun 4, 2019 · 2 comments

Comments

@jfdona23
Copy link

jfdona23 commented Jun 4, 2019

Compiling PyCopy 1.11 with the suggested ESP-IDF branch I get this error: unknown type name 'ppp_pcb'

Seems to be happening in Loboris fork too: loboris/ESP32-PPPOS-EXAMPLE#15

So far, I can skip this error by adding this line to Makefile:

INC += -I$(TOP)/lib/lwip/src/include

But now I've another error:

In file included from network_ppp.c:39:0:
../../lib/lwip/src/include/lwip/sockets.h:428:2: error: #error LWIP_SOCKET_OFFSET does not work with external FD_SET!
#error LWIP_SOCKET_OFFSET does not work with external FD_SET!

Fo the last error, I commented the line 408 in $(TOP)/lib/lwip/src/include/lwip/sockets.h

/* FD_SET used for lwip_select */ // LINE 407
//#ifndef FD_SET // LINE 408

...and added these in the next line (409):

#ifdef LWIP_SOCKET_OFFSET
#undef FD_SET
#undef FD_CLR
#undef FD_ISSET
#undef FD_ZERO
#undef _types_fd_set
#undef fd_set

Source: espressif/esp-idf#1141

But again, I got another error:

../../lib/lwip/src/include/lwip/arch.h:125:19: error: conflicting types for 'mem_ptr_t'
typedef uintptr_t mem_ptr_t;

All of this seems to be related to esp-idf since the change constantly. I'm working in the suggested branch for PyCopy 1.1: 5c88c5996

Any help will be appreciated :)

👋

@jfdona23
Copy link
Author

jfdona23 commented Jun 4, 2019

Posted in the correct repo - Sorry u.u

pfalcon/pycopy#31

@jfdona23 jfdona23 closed this as completed Jun 4, 2019
@pfalcon
Copy link
Owner

pfalcon commented Jun 4, 2019

Thanks for reposting to the correct project.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants