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

build broken because errors in esp-idf #1881

Closed
ataweg opened this issue Apr 24, 2018 · 2 comments
Closed

build broken because errors in esp-idf #1881

ataweg opened this issue Apr 24, 2018 · 2 comments

Comments

@ataweg
Copy link

ataweg commented Apr 24, 2018

Error is introduced with commit e37ceae
commit db5f927 compiles fine.
from file C:\Espressif\esp-idf\components\heap\multi_heap_platform.h #include <freertos/task.h> is removed and this caused the build failure.

Compile log:
In file included from C:/Espressif/esp-idf/components/heap/multi_heap_poisoning.c:27:0:
C:/Espressif/esp-idf/components/heap/multi_heap_platform.h:66:32: error: unknown type name 'TaskHandle_t'
#define MULTI_HEAP_BLOCK_OWNER TaskHandle_t task;
^
C:/Espressif/esp-idf/components/heap/multi_heap_poisoning.c:50:5: note: in expansion of macro 'MULTI_HEAP_BLOCK_OWNER'
MULTI_HEAP_BLOCK_OWNER
^
C:/Espressif/esp-idf/components/heap/multi_heap_poisoning.c: In function 'poison_allocated_region':
C:/Espressif/esp-idf/components/heap/multi_heap_platform.h:67:57: error: implicit declaration of function 'xTaskGetCurrentTaskHandle' [-Werror=implicit-function-declaration]
#define MULTI_HEAP_SET_BLOCK_OWNER(HEAD) (HEAD)->task = xTaskGetCurrentTaskHandle()
^
C:/Espressif/esp-idf/components/heap/multi_heap_poisoning.c:71:5: note: in expansion of macro 'MULTI_HEAP_SET_BLOCK_OWNER'
MULTI_HEAP_SET_BLOCK_OWNER(head);
^
C:/Espressif/esp-idf/components/heap/multi_heap_poisoning.c: In function 'multi_heap_get_block_owner':
C:/Espressif/esp-idf/components/heap/multi_heap_platform.h:68:42: warning: return makes pointer from integer without a cast [-Wint-conversion]
#define MULTI_HEAP_GET_BLOCK_OWNER(HEAD) ((HEAD)->task)
^
C:/Espressif/esp-idf/components/heap/multi_heap_poisoning.c:284:12: note: in expansion of macro 'MULTI_HEAP_GET_BLOCK_OWNER'
return MULTI_HEAP_GET_BLOCK_OWNER((poison_head_t*)multi_heap_get_block_address_impl(block));
^
cc1.exe: some warnings being treated as errors
make[1]: *** [/c/Espressif/esp-idf/make/component_wrapper.mk:286: multi_heap_poisoning.o] Fehler 1

@negativekelvin
Copy link
Contributor

@mahavirj
Copy link
Member

@ataweg Thanks for reporting this, please expect fix for this in some time.

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

3 participants