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

Port stripped on white space (e.g., [Out] Speaker) #87

Merged

Conversation

fabian-thomas
Copy link
Contributor

@fabian-thomas fabian-thomas commented May 8, 2024

When the port is something like "[Out] Speaker" the previous awk call only extracted "[Out]". Now it extracts all fields. I'm not sure if the modified awk call is the best one can use but it works.

Fixes #83

@nchachereau
Copy link
Contributor

I had recently found a better solution (and was going to submit it before I saw yours): setting a different field separator (": " instead of simply whitespace), using the following modified awk call:

awk -F ": " '/Active Port: / {print $2}'

Maybe you could update your pull request?

This would fix #83.

@fabian-thomas
Copy link
Contributor Author

Yep, let's do that. I added you as co-author.

@nchachereau
Copy link
Contributor

Oh that's kind! Thank you!

Copy link
Owner

@marioortizmanero marioortizmanero left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution everyone! Merging the PR :)

When the port is something like "[Out] Speaker" the previous awk call
only extracted "[Out]".

Co-authored-by: Nicolas Chachereau <nicolas@nchachereau.ch>
@marioortizmanero marioortizmanero merged commit dd6f72c into marioortizmanero:master May 18, 2024
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Port names containing whitespace are not supported
3 participants