[riot-notifications] [RIOT-OS/RIOT] Makefile.features: add declarative FEATURES_ variables definition (#11492)
notifications at github.com
Fri May 24 11:48:52 CEST 2019
cladmi commented on this pull request.
> @@ -1,3 +1,33 @@
# Process FEATURES variables
+# Resolve FEATURES_ variables
+# Their value will only be complete after resolving dependencies
+# Features that are required by the application but not provided by the BSP
+# Having features missing may case the build to fail.
+FEATURES_MISSING = $(sort $(filter-out $(FEATURES_PROVIDED),$(FEATURES_REQUIRED)))
+# Features that are only optional and not required at the same time.
+# The policy is to use by default whenever they are provided by the BSP.
+FEATURES_OPTIONAL_ONLY = $(sort $(filter-out $(FEATURES_REQUIRED),$(FEATURES_OPTIONAL)))
+FEATURES_OPTIONAL_USED = $(sort $(filter $(FEATURES_PROVIDED),$(FEATURES_OPTIONAL_ONLY)))
+# Optional features that will not be used because they are unavailable
+FEATURES_OPTIONAL_UNUSED = $(sort $(filter-out $(FEATURES_PROVIDED),$(FEATURES_OPTIONAL_ONLY)))
So you would prefer two variables for this.
Currently there would be no `FEATURES_OPTIONAL_UNUSED` as none of the two `DISABLE` or `BLACKLIST` are there.
I will change the `UNUSED` to go back to the `MISSING` behavior and see for a nice more describing comment.
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