[riot-notifications] [RIOT-OS/RIOT] sys/arduino: extension to Arduino module (#10592)

Gaëtan Harter notifications at github.com
Thu Sep 5 11:13:02 CEST 2019


Is the following the behavior your want?

If somebody wants the sketch, it does not use your lib.
If somebody wants your lib, it does not use the sketch.
Both sketch and lib depend on a "common" arduino part.

If this is the case, I think the current `arduino` module should be split between the sketch and the common that you also use.

So either rename it to `arduino_sketches`, and when an application that wants it, should say `USEMODULE += arduino_sketches`, and put the common as `arduino`.
Or it could be `arduino` == sketches, `arduino_lib` your thing, `arduino_common` common things required by both (if needed).

That depends on what is wanted to give as an interface to users.

Currently there was an automatic `arduino` is implemented by `arduino_sketches` that was done in the `sys/arduino/Makefile.include` and this goes against your need as it should be defined in `Makefile.dep` if keeping the current naming.

Depending on the naming wanted, I can do the change.

-- 
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/10592#issuecomment-528275592
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190905/236d13be/attachment.htm>


More information about the notifications mailing list