[riot-notifications] [RIOT-OS/RIOT] Makefile.include: add BUILD_FILES variable that holds all files to be linked (#12302)

Gaëtan Harter notifications at github.com
Wed Sep 25 16:23:21 CEST 2019


> > This actually still has some issue when compiling riotboot target
> 
> This would need something like...
> 
> ```
> ifeq ($(BUILD_IN_DOCKER),1)
> # It will trigger executing 'scan-build' or 'scan-build-analyze' with docker
> riotboot: ..in-docker-container
> riotboot/slot0: ..in-docker-container
> riotboot/slot1: ..in-docker-container
> riotboot/combined-slot0: ..in-docker-container
> endif # BUILD_IN_DOCKER
> ```
> 
> This isn't in the scope of the original issue @cladmi? right?

Only few targets are mapped to be build in docker in RIOT, somehow only 'all' or similar. To add support they must be added to

https://github.com/RIOT-OS/RIOT/blob/0170d1c5da244a17c66cd1708d446eb4367e7695/makefiles/docker.inc.mk#L6-L12

So this is indeed out of the scope of my original issue.

Doing `mcuboot-flash` would also not currently work.

-- 
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/12302#issuecomment-535047730
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190925/d4d2d8b6/attachment.htm>


More information about the notifications mailing list