[riot-notifications] [RIOT-OS/RIOT] cpu/kinetis: fix values stored in ROM_LEN/RAM_LEN variables (#10888)

Alexandre Abadie notifications at github.com
Mon Feb 4 09:05:26 CET 2019


> having to add a subshell here for an unknown reason

The problem is because we need to perform the variable expansion before the replacement that is done on it for computing the SLOT0_LEN [here](https://github.com/RIOT-OS/RIOT/blob/master/cpu/cortexm_common/Makefile.include#L35). This [SO answer](https://stackoverflow.com/a/15978322) mention that the `$$` requires this expansion to be done by the shell.

> Does it work if you just remove the spaces inside the parentheses and keep the whole thing as it is in master?

I tried and it doesn't make any difference. Another potential fix is to use the shell command when evaluating the value of ROM_LEN [here](https://github.com/RIOT-OS/RIOT/blob/master/cpu/cortexm_common/Makefile.include#L35).

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/10888#issuecomment-460160117
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190204/7267d24c/attachment-0001.html>


More information about the notifications mailing list