[riot-notifications] [RIOT-OS/RIOT] sys/Makefile.include: include riotboot headers when FEATURES_REQUIRED=riotboot (#11566)

Kaspar Schleiser notifications at github.com
Thu May 23 14:57:25 CEST 2019


kaspar030 commented on this pull request.



> @@ -77,7 +77,7 @@ ifneq (,$(filter printf_float,$(USEMODULE)))
   endif
 endif
 
-ifneq (,$(filter riotboot_%,$(USEMODULE)))
+ifneq (,$(filter riotboot%,$(FEATURES_USED)))

I think the ```%``` at the end of riotboot is not necessary. If you feel like, could you confirm and then force-push its removal? I'm OK with merging as is, too. ;)

-- 
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/11566#pullrequestreview-241183344
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190523/33834e0a/attachment.html>


More information about the notifications mailing list