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

FEATURE_CLIENT compilation fails #2569

Closed
tommikas opened this issue Aug 29, 2016 · 2 comments
Closed

FEATURE_CLIENT compilation fails #2569

tommikas opened this issue Aug 29, 2016 · 2 comments

Comments

@tommikas
Copy link
Contributor

Compile: m2mconnectionhandler.cpp
Compile: m2mtimer.cpp
Compile: m2mtimerpimpl.cpp
Compile: m2mconnectionhandlerpimpl.cpp
[Error] m2mconnectionhandlerpimpl.cpp@41,8: 'MemoryPool' does not name a type
[Error] m2mconnectionhandlerpimpl.cpp@84,9: 'memory_pool' was not declared in this scope
[Error] m2mconnectionhandlerpimpl.cpp@159,28: 'memory_pool' was not declared in this scope
[Error] m2mconnectionhandlerpimpl.cpp@266,28: 'memory_pool' was not declared in this scope
[Error] m2mconnectionhandlerpimpl.cpp@328,28: 'memory_pool' was not declared in this scope
[ERROR] ./mbed-os/features/FEATURE_CLIENT/mbed-client-classic/source/m2mconnectionhandlerpimpl.cpp:41:8: error: 'MemoryPool' does not name a type
 static MemoryPool<M2MConnectionHandlerPimpl::TaskIdentifier, MBED_CLIENT_EVENT_LOOP_SIZE/64> memory_pool;
        ^
./mbed-os/features/FEATURE_CLIENT/mbed-client-classic/source/m2mconnectionhandlerpimpl.cpp: In function 'void connection_tasklet_event_handler(arm_event_s*)':
./mbed-os/features/FEATURE_CLIENT/mbed-client-classic/source/m2mconnectionhandlerpimpl.cpp:84:9: error: 'memory_pool' was not declared in this scope
         memory_pool.free(task_id);
         ^

Since #2527 was merged in, because of https://github.com/ARMmbed/mbed-os/pull/2527/files#diff-d62a3ea84a2a42fc1bf16f8fa83b4725L30

@yogpan01
Copy link
Contributor

This is because of 2 merges which missed the include header file.
There needs new PR from mbed-client-classic to resolve this issue. I will create a new PR now to get this resolved.

yogpan01 pushed a commit to yogpan01/mbed that referenced this issue Aug 29, 2016
This commit fixes issue
ARMmbed#2569

This commit has to go in coming mbed-os release else the build is broken.
0xc0170 pushed a commit that referenced this issue Aug 30, 2016
This commit fixes issue
#2569

This commit has to go in coming mbed-os release else the build is broken.
@tommikas
Copy link
Contributor Author

Fixed in #2570, so closing this.

0xc0170 pushed a commit that referenced this issue Aug 31, 2016
This commit fixes issue
#2569

This commit has to go in coming mbed-os release else the build is broken.
miklis pushed a commit that referenced this issue Sep 12, 2016
This commit fixes issue
#2569

This commit has to go in coming mbed-os release else the build is broken.
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