-
Notifications
You must be signed in to change notification settings - Fork 67
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
Support switching mesh radio on multi-radio devices #847
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See comment about missing "=" in the test on line 173. Once that is added, the feature works as designed. Thanks very much, @aanon4
files/usr/local/bin/node-setup
Outdated
@@ -170,7 +170,9 @@ do | |||
end | |||
|
|||
if cfg.wifi_enable == "1" then | |||
cfg.wifi_intf = aredn.hardware.get_board_network_ifname("wifi"):match("^(%S+)") | |||
if not cfg.wifi_intf or cfg.wifi_intf = "" then |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is a missing "=" in the if statement on line 173.
Should be if not cfg.wifi_intf or cfg.wifi_intf == "" then
Once that is added, the rest of the feature works as designed.
Also, for some reason the new Mesh RF link is not showing up on the Neighbor Status page |
@ab7pa Nice catch, thanks. Fix it. |
@ab7pa Thanks - looks like there are a few more places I need to review for where the default mesh radio is hard coded. I'll keep this in draft until I'm done with that. |
BTW did I mention that this is an AWESOME FEATURE! :) |
Okay, backup for review after a few fixes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perfect! That fixes all the issues. Thanks very much, @aanon4 Tim!
No description provided.