[riot-notifications] [RIOT-OS/RIOT] makefiles/docker: add flash and term in docker (#11220)

Gaëtan Harter notifications at github.com
Wed Mar 20 17:07:32 CET 2019


That is an idea I had some time ago also. Even somehow be able to run tests completely from within docker.

It makes sense for example with `native` that you cannot always run on your machine when build in docker. (not exactly the same system, running on mac), or some toolchains that are a pain to install.
Also, even if the build system builds `edbg` you still need the libusb-dev/libhidapi-dev.
Having a slower but reliable way to build in docker is a good thing too.

I am also thinking about cases where the tests need to be run as `root` to be run from a container.

But it will be more complicated to make it work correctly and not only working for a few boards.
So this would be more a tracking PR thing than a 2 commit solution:

Problem I already see:

* the current docker image is a builder docker image, so does not have any tools (openocd/JLINK) installed inside. So it would require a second docker image. It is a good thing to do I think. This would however maybe force to have a different recipe for this one and so the complications…
* Some board have different PROGRAMMER_PORT and UART_PORT. It is not explicit right now but both should be made explicit.
* edbg.inc.mk does not use `PORT` for flashing but only `DEBUG_ADAPTER_ID` so they must be in sync.


The `PROGRAMMER_PORT/UART_PORT` and no harmony with `DEBUG_ADAPTER_ID/SERIAL/JLINK_SERIAL` handling are things that I also need and would happily follow and help working on the issue.

So for me it is too early to say that something like this can be merged but is something I am interested in.

-- 
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/11220#issuecomment-474904889
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190320/a36f44d4/attachment.html>


More information about the notifications mailing list