frc1418 / de Goto Github PK

Having people set a bunch of environment variables is an impractical way to handle configuration. It might be better if we store all the configuration information in a dotfile in root or something like that.

We might want or need to use these as an alternative to the Thunderbolt-ethernet adapters.

As much as bash scripts are fun, we may want to consider a rewrite in Python or another language. It would be a complicated process, but it might help with some other issues.

For this, you will likely need to find a way to iterate through each variable, ask the user for the value, and store it. Alternatively (perhaps this would be easier), you could just cat an empty config into ~/.de and open it in the user's $EDITOR for them to write to manually. Choose whatever you think is best.

@AndrewLester

https://serverfault.com/questions/52034/what-is-the-difference-between-double-and-single-square-brackets-in-bash

Currently it is still "dep.7"

Right now make puts the program into /usr/local/bin. This is common practice, but that directory by default requires users to have root/sudo access.

A better option would be to put it in ~/.local/bin, which is writable without root (by the owner of the home directory, of course).

The issue with this is that the $PATH variable doesn't contain that directory by default on most systems. We'd need to check that that directory is in the path and warn the user if necessary that they need to add it.

This is a way we could potentially make things more generalized than naming enX every time. airport will automatically give you the WiFi device.

There may be a name for Ethernet (-over-Thunderbolt) as well, gotta look into that.

It just feels sort of off to ask for passwords first.

This tool has a naming conflict with golang/dep.

The content is on the manpage right now.

We still use names formatted as DE_WHATEVER_THE_NAME_IS. There are two things wrong with this:

• Firstly, we're now using a config file, so everything in the file will obviously pertain to this application and we don't need to say DE_ for clarification at the starting of each name.
• Since they similarly aren't global environment variables anymore, they don't need to be capitalized.

We should just have another environment variable (DEPLOY_ROBOT_NETWORK_SSID, probably) rather than hardcoding it to 1418.

Right now we reconnect to whichever network we started on, but try to use the same PSK no matter what. This works fine for me since I'm usually at school and only need to reconnect to FCCPS-STUDENT, but it's not a great implementation.

It's not a great idea to make users store passwords for their networks in environment variables or have to handle them at all. We should at least handle encrypted passwords.

I created a tool called wif over the summer, which handles network switching with ease. Supporting switching with that might make things easier in terms of credential management and the like.

https://github.com/ErikBoesen/wif