Skip to content

Changed nmcli cmd to check on AP connect to using '*' in the in-use f…#365

Merged
senorprogrammer merged 1 commit intowtfutil:masterfrom
jdenoy:master
Jan 30, 2019
Merged

Changed nmcli cmd to check on AP connect to using '*' in the in-use f…#365
senorprogrammer merged 1 commit intowtfutil:masterfrom
jdenoy:master

Conversation

@jdenoy
Copy link
Contributor

@jdenoy jdenoy commented Jan 30, 2019

Changed nmcli cmd to check on AP connect to using '*' in the in-use field instead of 'yes/no' from the active field. This is needed for non english language set in operating system.

This is needed because yes/no will not be the same text depending on the language configured in the operating system. Using the "in-use" field that is either set to "*" ot "" will be easier for the "wtf.FindMatch" function to match.

…ield instead of 'yes/no' from the active field. This is needed for non english language set in operating system
@senorprogrammer
Copy link
Collaborator

Very nice, thanks for improving this.

@senorprogrammer senorprogrammer merged commit ab0dde5 into wtfutil:master Jan 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants