search

fAuernigg / esp-baresip

A baresip component of esp-idf for esp32.
BSD 3-Clause "New" or "Revised" License
7 stars 4 forks source link

Compatibility with ESP-IDF 3.3 (stable) #2

Open sosaucily opened 4 years ago

sosaucily commented 4 years ago

When compiling this package in an esp-idf/adf project using the 3.3 version of idf, errors occur with the SoC component of the esp-idf library.

For example:

esp-idf/components/soc/esp32/include/soc/cpu.h:36:5: error: 'asm' undeclared (first use in this function)
     asm volatile ("mov %0, sp;" : "=r" (sp));

3.3 is the stable build of esp-idf. Not sure why this is happening, but i'll dig in a bit today.

sosaucily commented 4 years ago

Actually, it seems the flag -std=c99 in the component.mk file is causing this, as discussed here: https://stackoverflow.com/questions/35131350/error-asm-undeclared-first-use-in-this-function

Without this std=c99 flag it compiles with esp-idf 3.3. Can I make a PR?

sosaucily commented 4 years ago

I also had trouble with this flag, and needed to comment it out in order to remove some overwhelming number of warnings during compilation.

CFLAGS += -Wsign-compare

fAuernigg commented 4 years ago

Yes please make a PR. Also ignoring a few warnings sounds good. Especially Wsign-compare sounds ok to me.