[riot-notifications] [RIOT-OS/RIOT] makefiles/riotboot: ensure BUILDDEPS are built before riotboot targets (#15860)

Leandro Lanzieri notifications at github.com
Tue Jan 26 18:21:13 CET 2021


### Contribution description
As reported in #15434, there seems to be a race condition when building applications with riotboot with STM32 targets, because of the cloning of CMSIS. This adds `BUILDDEPS` as a dependency for the `riotboot/bootloader/%` targets, so we ensure that all the needed files are there before start building the riotboot binary. This prevents starting the clone of CMSIS twice at the same time, which seems to be the problem reported in #15434.

### Testing procedure
- Follow the steps described in #15434, in master you should (after some tries) fall into the race condition. This PR should fix this problem.

### Issues/PRs references
Fixes #15434
You can view, comment on, or merge this pull request online at:

  https://github.com/RIOT-OS/RIOT/pull/15860

-- Commit Summary --

  * makefiles/riotboot: ensure BUILDDEPS are built before riotboot targets

-- File Changes --

    M makefiles/boot/riotboot.mk (2)

-- Patch Links --

https://github.com/RIOT-OS/RIOT/pull/15860.patch
https://github.com/RIOT-OS/RIOT/pull/15860.diff

-- 
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/15860
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210126/23f03202/attachment.htm>


More information about the notifications mailing list