[riot-notifications] [RIOT-OS/RIOT] cpu/kinetis: fix values stored in ROM_LEN/RAM_LEN variables (#10888)
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:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the notifications