[riot-commits] [RIOT-OS/RIOT] 63c1a4: makefiles/tools/programmer: do not use programmer ...

Kaspar Schleiser noreply at github.com
Mon Apr 26 21:17:51 CEST 2021


  Branch: refs/heads/2021.04-branch
  Home:   https://github.com/RIOT-OS/RIOT
  Commit: 63c1a45e1709945ecc3388057c7501c24e20131d
      https://github.com/RIOT-OS/RIOT/commit/63c1a45e1709945ecc3388057c7501c24e20131d
  Author: Benjamin Valentin <benpicco at beuth-hochschule.de>
  Date:   2021-04-26 (Mon, 26 Apr 2021)

  Changed paths:
    M makefiles/tools/programmer.inc.mk

  Log Message:
  -----------
  makefiles/tools/programmer: do not use programmer wrapper

While nice at first, this causes more issues than it's worth for
a slightly more pretty output.

Hiding information from developers is not a good idea, especially when
it comes to an often finicky programming step.

While in theory error output should be caught, there are enough
cases where the programmer will get stuck in a loop or require user
input that hiding programmer output by default is a bad idea.

(cherry picked from commit a0b83f8195c4cc0a87f5bbf605e3d88ad3defe09)


  Commit: bebf2c0ed387190ba4d70488f7928b3d4a178141
      https://github.com/RIOT-OS/RIOT/commit/bebf2c0ed387190ba4d70488f7928b3d4a178141
  Author: Kaspar Schleiser <kaspar at schleiser.de>
  Date:   2021-04-26 (Mon, 26 Apr 2021)

  Changed paths:
    M makefiles/tools/programmer.inc.mk

  Log Message:
  -----------
  Merge pull request #16390 from kaspar030/backport/2021.04/makefiles/tools/programmer-disable

tools/programmer: do not use programmer wrapper [backport 2021.04]


Compare: https://github.com/RIOT-OS/RIOT/compare/80d5f8fb2733...bebf2c0ed387


More information about the commits mailing list