[riot-notifications] [RIOT-OS/RIOT] usbus: Add automated test (#12267)

Martine Lenders notifications at github.com
Tue Sep 17 22:58:42 CEST 2019


miri64 commented on this pull request.



> +USEMODULE += embunit
+USEMODULE += usbus
+FEATURES_PROVIDED += periph_usbdev
+
+# USB device vendor and product ID
+# pid.codes test VID/PID, not globally unique
+USB_VID ?= 1209
+USB_PID ?= 0001
+
+CFLAGS += -DUSB_CONFIG_VID=0x$(USB_VID) -DUSB_CONFIG_PID=0x$(USB_PID)
+
+include $(RIOTBASE)/Makefile.include
+
+.PHONY: usb_id_check
+usb_id_check:
+	@if [ $(USB_VID) = $(DEFAULT_VID) -o $(USB_PID) = $(DEFAULT_PID) ] ; then \

> […] and what's wrong with ~bashing~ (oups ;) ) bash-ism.

~~One dependency less~~ (ok since we use already plenty of them this argument is invalid) and assuming that bash is the default shell (which it is often, but not always ;-)).

-- 
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/12267#discussion_r325382805
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190917/af1a9ec6/attachment.htm>


More information about the notifications mailing list