Skip to content

makefiles/tools/serial.inc.mk: Handle new miniterm versions [backport 2023.01]#19451

Merged
bors[bot] merged 1 commit intoRIOT-OS:2023.01-branchfrom
MrKevinWeiss:backport/2023.01/pr/fix/miniterm
Apr 4, 2023
Merged

makefiles/tools/serial.inc.mk: Handle new miniterm versions [backport 2023.01]#19451
bors[bot] merged 1 commit intoRIOT-OS:2023.01-branchfrom
MrKevinWeiss:backport/2023.01/pr/fix/miniterm

Conversation

@MrKevinWeiss
Copy link
Copy Markdown
Contributor

Backport of #19444

Contribution description

While testing examples/micropython I notice that the default of miniterm.py is actually miniterm. To simplify user setups, this checks for miniterm.py first then falls back to miniterm.

Testing procedure

Take any board with any newish version of Ubuntu and run

make -C flash test examples/micropython

If you have miniterm.py in PATH or if it is miniterm both should work.

Issues/PRs references

While testing examples/micropython I notice that the default of miniterm.py is actually miniterm.
To simplify user setups, this checks for miniterm.py first then falls back to miniterm.

(cherry picked from commit 6d72903)
@MrKevinWeiss MrKevinWeiss added Area: build system Area: Build system Area: tools Area: Supplementary tools CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Process: release backport Integration Process: The PR is a release backport of a change previously provided to master Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) labels Apr 4, 2023
@riot-ci
Copy link
Copy Markdown

riot-ci commented Apr 4, 2023

Murdock results

✔️ PASSED

a9968ea makefiles/tools/serial.inc.mk: Handle new miniterm versions

Success Failures Total Runtime
6770 0 6770 11m:05s

Artifacts

@maribu
Copy link
Copy Markdown
Member

maribu commented Apr 4, 2023

bors merge

@bors
Copy link
Copy Markdown
Contributor

bors bot commented Apr 4, 2023

Build succeeded:

@bors bors bot merged commit 6fb5d0c into RIOT-OS:2023.01-branch Apr 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: build system Area: Build system Area: tools Area: Supplementary tools CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Process: release backport Integration Process: The PR is a release backport of a change previously provided to master Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants