Skip to content
This repository has been archived by the owner on Sep 20, 2022. It is now read-only.

sniffer: correct handling of byte strings #81

Merged
merged 2 commits into from
Sep 20, 2022
Merged

sniffer: correct handling of byte strings #81

merged 2 commits into from
Sep 20, 2022

Conversation

miri64
Copy link
Member

@miri64 miri64 commented Sep 19, 2022

@benpicco benpicco requested a review from chrysn September 19, 2022 16:33
chrysn
chrysn approved these changes Sep 19, 2022
View reviewed changes
Copy link
Member

@chrysn chrysn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. The shebang could be changed to python3 (now that it works with both, and even when systems do still have Python2 installed, they may not have python2-serial).

@miri64
Copy link
Member Author

miri64 commented Sep 20, 2022

The shebang could be changed to python3

Done

@miri64 miri64 merged commit 4de0fce into RIOT-OS:master Sep 20, 2022
@miri64 miri64 deleted the sniffer/fix/python3 branch September 20, 2022 07:58
chrysn pushed a commit to chrysn-pull-requests/RIOT that referenced this pull request Sep 20, 2022
@miri64 @chrysn
Merge pull request RIOT-OS/applications#81 from miri64/sniffer/fix/py…
a6a642a 
…thon3

sniffer: correct handling of byte strings
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@chrysn chrysn chrysn approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@miri64 @chrysn