SSID List for Wireless Windows machines - Githubissues

cloudmesh / cloudmesh-pi-burn

Burns many SD cards so we can build a Raspberry PI cluster

Other

18 stars 12 forks source link

SSID List for Wireless Windows machines #62

Closed dkkolli closed 2 years ago

dkkolli commented 2 years ago

Give users list of ssids for windows users to choose from if ssid is not entered initially in cms burn raspberry command

Could not get Printer.write() to work instead of printing list.

dkkolli commented 2 years ago

Updating for wired connections also

laszewsk commented 2 years ago

HOw are you able to update your pull request into cloudmesh? https://github.com/cloudmesh/cloudmesh-pi-burn/pull/62 I like to identify if the pull request you made is done by you or done in some other way. As far as i can tell you do not have right to merge pull requests, but I may be wrong ...

dkkolli commented 2 years ago

If using a wireless windows machine, provides list of wifi networks: +-------+-----------------+ | Index | Host | +-------+-----------------+ | 0 | eduroam | | 1 | iPhone | | 2 | IU Guest | | 3 | IU Secure | | 4 | GNXS-9959 | | 5 | Bluegreen_GUEST | | 6 | ZyXEL7A2C00 | | 7 | MetroDK | | 8 | 2WIRE131 | +-------+-----------------+ Enter ssid from list:

If using a wired windows machine, lets user know machine is not configured for wifi.

laszewsk commented 2 years ago

I think on osx we do not need a list we simply look up what the wifi is that the laptop uses. is ther not just such a function. otherwise shoudl we not implement a function

list_wifi() and if the current wifi cfrom the laaptop can not pe obtained in windows we use the list function ... so i would split this functionality.

laszewsk commented 2 years ago

JP will check and review the newest version of the code.

[x] I think we implemented the ssid find
[x] i think we implemented country code
[x] I suggested to implement cloudmesh.common.util.getpass so it works on all architectures including WIndows. this way we can use this getpass and not the one from python as its documented the windows version is not implementing getpass

laszewsk commented 2 years ago

there are conflicts

laszewsk commented 2 years ago

I suggest not to modify this pull request, but to just do its completely from scratch and than we just close this pull requests much easier