Skip to content

Conversation

nuket
Copy link

@nuket nuket commented Aug 17, 2016

Without this .mbedignore file, the bulk of the ArduinoJson test/* code gets pulled into the mbed OS build process, which causes the final firmware to be too large for the target processor, since most off-the-shelf embedded processors have somewhere between 128 - 256KB of flash.

When running mbed compile without this file, the linker errors out unhappily:

./mbed-os/hal/targets/cmsis/TARGET_NORDIC/TARGET_NRF5/TARGET_MCU_NRF51822_UNIFIED/TOOLCHAIN_GCC_ARM/TARGET_MCU_NORDIC_32K/NRF51822.ld:142 cannot move location counter backwards (from 0000000020007fe0 to 0000000020007800)
collect2: error: ld returned 1 exit status
[ERROR] ./mbed-os/hal/targets/cmsis/TARGET_NORDIC/TARGET_NRF5/TARGET_MCU_NRF51822_UNIFIED/TOOLCHAIN_GCC_ARM/TARGET_MCU_NORDIC_32K/NRF51822.ld:142 cannot move location counter backwards (from 0000000020007fe0 to 0000000020007800)
collect2: error: ld returned 1 exit status

With the .mbedignore file, everything is fine.

I also reported this issue as - ARMmbed/mbed-os#2467

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 4209561 on nuket:master into * on bblanchon:master*.

@bblanchon
Copy link
Owner

Hi Max,

Thank you very much for this contribution 👍

I think that this file should be in the root folder, along with .gitignore.

Also, I think you should also add other folders:

/.github/
/scripts/
/tests/
/third-party/

And probably unwanted files in the root folder.

Would that work?

Regards,
Benoit

@bblanchon
Copy link
Owner

@nuket Did you see my message above?

@nuket
Copy link
Author

nuket commented Aug 23, 2016

Yes, I will try this out.

On August 23, 2016 15:04:49 Benoît Blanchon notifications@github.com wrote:

@nuket Did you see my message above?

You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
#334 (comment)

@nuket
Copy link
Author

nuket commented Aug 24, 2016

Ok, I've added the folders above, looks good.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling fcb8454 on nuket:master into * on bblanchon:master*.

bblanchon pushed a commit that referenced this pull request Aug 25, 2016
@bblanchon
Copy link
Owner

Manually merged in a7ef99d

@bblanchon bblanchon closed this Aug 25, 2016
Repository owner locked and limited conversation to collaborators Sep 21, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants