Testing

[alexAubin]

No description provided.

[Thatoo]

Could it be merged?
Since update to v2.0, only the internal wifi of the raspberry pi is working as hotspot. The panda usb wireless dongle is unfortunately useless...

[alexAubin]

We should be careful about other places where iw_devices is invoked to be consistent:

 $ grep -nr iw_devices ./
./scripts/_common.sh:15:function iw_devices()
./scripts/config:30:    if [[ $(iw_devices) == "" ]]
./scripts/config:79:    if [[ $(iw_devices) == "" ]]
./scripts/config:86:        for device in $(iw_devices | sed "s/,/ /g")
./scripts/install:133:wifi_device=$(iw_devices | awk -F\| '{ print $1 }')
./scripts/restore:106:wifi_device=$(iw_devices | awk -F\| '{ print $1 }')
./scripts/upgrade:167:wifi_device=$(iw_devices | awk -F\| '{ print $1 }')

@zamentur : do you know why in install/restore/upgrade we seem to be using | as a separator (which indeeds seems to corresponds to the code of iw_devices, but this PR assumes the separator to be , ?

[alexAubin]

@Thatoo : could you share the output of /sbin/iw dev | grep Interface ?

[Thatoo]

sure! Thank you Alex for taking some of your time to help me.

~ $ /sbin/iw dev | grep Interface
	Interface wlan1
	Interface wlan0

which seems to say that both devices are detected. If I disconnect the wireless usb dongle, only wlan0 remain.

However, when I go to hotspot settings, I don't have any choice, only one option with wlan1|wlan0 as explain in picture in here : #91

[Thatoo]

We should be careful about other places where iw_devices is invoked to be consistent:

 $ grep -nr iw_devices ./
./scripts/_common.sh:15:function iw_devices()
./scripts/config:30:    if [[ $(iw_devices) == "" ]]
./scripts/config:79:    if [[ $(iw_devices) == "" ]]
./scripts/config:86:        for device in $(iw_devices | sed "s/,/ /g")
./scripts/install:133:wifi_device=$(iw_devices | awk -F\| '{ print $1 }')
./scripts/restore:106:wifi_device=$(iw_devices | awk -F\| '{ print $1 }')
./scripts/upgrade:167:wifi_device=$(iw_devices | awk -F\| '{ print $1 }')

@zamentur : do you know why in install/restore/upgrade we seem to be using | as a separator (which indeeds seems to corresponds to the code of iw_devices, but this PR assumes the separator to be , ?

You're awesome @alexAubin !!!!! with | it works!
line 86 should not be changed from
for device in $(iw_devices)
to
for device in $(iw_devices | sed "s/,/ /g")
but to
for device in $(iw_devices | sed "s/|/ /g")

[ericgaspar]

!testme

[yunohost-bot]

🚀