[riot-notifications] [RIOT-OS/RIOT] examples/cord_epsim: change default RD server addr handling (#10464)

Martine Lenders notifications at github.com
Wed Mar 27 11:19:36 CET 2019


miri64 commented on this pull request.



> +# If nothing else is defined, we use CoAP default port
+RD_PORT ?= COAP_PORT
+CFLAGS += -DRD_PORT=$(RD_PORT)
+
+# The RD server's address must be defined by the build environment by setting
+# the RD_ADDR environment variable. Per default, this value is set to the
+# loopback address for enabling the build tests to successfully build this
+# example.
+RD_ADDR ?= \"::1\"
+CFLAGS += -DRD_ADDR=$(RD_ADDR)
+
+# In case the default value for RD_ADDR is used, we display a warning to the
+# user.
+ifeq (\"::1\", $(RD_ADDR))
+  ifneq (clean, $(MAKECMDGOALS))
+    $(info WARNING: the RD server address is set to the loopback device!)

Looks to me like it is this line that is causing problems on Murdock with its weird way to parse output, since `RD_ADDR` is set to `::1` above.

-- 
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/10464#pullrequestreview-219371279
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190327/702bc4e6/attachment.html>


More information about the notifications mailing list