Changes to Firmware Wizard for Arduino IDE

dillonledoux commented 4 years ago

List of VIs changed and why:

Firmware Wizard Config.vi: Added case structure so that if the copy cannot take place for whatever reason, skip the "Done" dialog box.
Verify Arduino Libraries.vi: This VI relies on the arduino_debug.exe to get the path for the libraries, therefore we need to make sure its present. If not, then alert the user that they don't have a current version of the IDE installed or they pointed to the wrong base path and export whether the IDE was found so the caller is aware. Also adjusted the search string for the command line path to better handle the IDE output.
Copy Firmware Libraries.vi: Changed False Constant to True on Open Config Data.vi so the ini file is created on first run and so that error 7 is not thrown

dillonledoux commented 4 years ago

Any updates on this?

spathiwa commented 4 years ago

We use an internal Perforce mirror of this repo internally, and I just realized that an owner of this code which is probably the right person to review this code isn't a member of the GitHub project. I've added him and hopefully he should be able to accept and review this shortly. @gmckaskle

sharpk commented 3 years ago

I tried this out and it seems to work well and improves error handling significantly. Nice work!

Here's what I tested:

Made sure I got a reasonable error from hitting the Generate button when no Arduino IDE is installed.
Uploaded firmware to an Arduino IDE using pre-compiled firmware
Uploaded firmware to an Arduino IDE by compiling firmware with Arduino IDE.

I'll see that this gets pulled here and pulled to our internal repo.

LVMakerHub / LINX

Changes to Firmware Wizard for Arduino IDE #116