NRF_LOG_STR_PUSH_BUFFER_SIZE defined twice

RE: NRF_LOG_STR_PUSH_BUFFER_SIZE defined twice

Christian Waldmann — Thu, 13 Sep 2018 12:16:33 GMT

The missing entry in my project is based on the fact that I am trying to upgrade an existing project from SDK 5.0 to SDK 5.2. So in the already existing sdk_config.h in my project the entry is missing.

Regarding the many changes in the source tree, this way of porting is hard to to.

I think I will try to integrate my solution again in the example from SDK 5.2 that was my starting point in the SDK 5.0.

Happy coding

Chris

RE: NRF_LOG_STR_PUSH_BUFFER_SIZE defined twice

tesc — Thu, 13 Sep 2018 11:46:23 GMT

Hi,

Thank you for the clarification.

Yes, both issues are reported in our internal issue tracker as separate bugs.

It currently is not entirely clear what posts are replies to what other posts here on DevZone, but the post about "sections of the sdk_config.h file missing" was to Christian Waldmann and the post about "too late in the release pipeline" was to you (gnf).

Regards,
Terje

RE: NRF_LOG_STR_PUSH_BUFFER_SIZE defined twice

gnf — Thu, 13 Sep 2018 07:39:47 GMT

Ok, now I see the last post was an answer to Christian. Anyway, to clarify even more, the problem I am talking about is that the section is defined twice, one after each other.

I attach two screenshots to explain this in detail.

Keil Configuration Wizard:

Here in a search in Eclipse. To me it seems a lot of sdk_config.h files suffer from this:

RE: NRF_LOG_STR_PUSH_BUFFER_SIZE defined twice

tesc — Wed, 12 Sep 2018 15:00:23 GMT

Hi,

We have had some issues with some sections of the sdk_config.h file missing in some examples (but still it was present in others). This is a known issue, and is probably the reason why you did not find it in the example that you were looking at. (It is not limited to NRF_LOG_STR_PUSH_BUFFER_SIZE.)

Regards,
Terje

RE: NRF_LOG_STR_PUSH_BUFFER_SIZE defined twice

tesc — Wed, 12 Sep 2018 14:58:12 GMT

Hi,

I am afraid nRF5 SDK v15.2.0 was too far in the release pipeline for going back to change this before release. (This is not really a critical bug, although it may be a bit annoying.) Hopefully we will look into it shortly and fix it for future releases.

Regards,
Terje

RE: NRF_LOG_STR_PUSH_BUFFER_SIZE defined twice

gnf — Wed, 12 Sep 2018 14:16:46 GMT

No. I'm sorry but I just downloaded and tested nRF5_SDK_15.2.0_9412b96. The problem is still there!

RE: NRF_LOG_STR_PUSH_BUFFER_SIZE defined twice

Christian Waldmann — Wed, 12 Sep 2018 13:29:09 GMT

Now the section isn't present anymore in the released nRF5_SDK_15.2.0_9412b96

When sdding it once, it compiles

RE: NRF_LOG_STR_PUSH_BUFFER_SIZE defined twice

tesc — Fri, 07 Sep 2018 14:52:58 GMT

Hi,

Thank you for notifying us about this bug. It is highly appreciated! I have registered it in our internal tracker. Hopefully we can fix it before too long.

Regards,
Terje