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

Juan I Carrano notifications at github.com
Wed Mar 20 15:58:12 CET 2019


The plugdev group is an "Ubuntuism", as far as I know, a weird invention because they did not  want to do permissions properly. Nowadays systemd sets ACLs for devices based on active console sessions:

```sh
$ getfacl /dev/ttyACM0
getfacl: Removing leading '/' from absolute path names
# file: dev/ttyACM0
# owner: root
# group: root
user::rw-
user:jcarrano:rw-
group::rw-
mask::rw-
other::---

```

I do not believe it is a good idea to add a feature that _forces_ users to subvert their machine's permission system. I understand the spirit of this kind of PRs that try to make some things easier for certain users, but I don't see that much of a benefit:

- SAM boards are flashed by edbg. Downloading and building this tools is handled by the build system already.
- `make term` does not normally require any hard-to-get tool (except if using jlink RTT, see below)
- jlink is required for flashing and debugging some boards and, if one is using RTT, for `make term` too. However, _I doubt we are able to redistribute it_.
- The only remaining issue is OpenOCD, which has no official releases and as such may be harder for the user to get an updated version in his machine.

On the last point, I believe it is more useful for the community to provide a PPA, if there isn't one already.

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


More information about the notifications mailing list