Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dist/tools/usb-serial: call ttys.py with its path [backport 2023.07] #19831

Conversation

maribu
Copy link
Member

@maribu maribu commented Jul 14, 2023

Backport of #19823

Contribution description

This PR provides a fix for #19805
When calling SERIAL=xxx make flash the command failed because the script dist/tools/usb-serial/find-ttys.sh tries to call ttys.py but it doesn't find it. So pass the full path to ttys.py to get rid of the problem.

Testing procedure

Connect at least two boards to your computer
run make list-ttys to get the ID of your boards
Try to flash one of them with (replace SERIAL with the one you get with the previous command):
BOARD=xxx SERIAL=xxx make -C tests/leds flash
It fails on master.
It should works with this PR.

Issues/PRs references

Fixes #19805

Signed-off-by: Dylan Laduranty <[email protected]>
(cherry picked from commit 3b85c56)
Signed-off-by: Dylan Laduranty <[email protected]>
(cherry picked from commit 1007f02)
@maribu maribu added 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 labels Jul 14, 2023
@riot-ci
Copy link

riot-ci commented Jul 14, 2023

Murdock results

✔️ PASSED

6b7e2c3 dist/tools/usb-serial/find-tty.sh: call ttys.py with its path

Success Failures Total Runtime
1 0 1 58s

Artifacts

@benpicco
Copy link
Contributor

bors merge

@bors
Copy link
Contributor

bors bot commented Jul 14, 2023

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

@bors bors bot merged commit e3e09f0 into RIOT-OS:2023.07-branch Jul 14, 2023
@benpicco benpicco added this to the Release 2023.07 milestone Aug 2, 2023
@maribu maribu deleted the backport/2023.07/dist/tools/usb-serial/fix_find-tty branch April 23, 2024 08:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants