Skip to content
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

[2.0.x] Faster disk find for LPC1768, Auto-build CDC support fix, error handling improvements #10849

Merged

Conversation

Bob-the-Kuhn
Copy link
Contributor

  1. upload_extra_script.py is faster and only executes if doing an upload (if run under Auto-build)
  2. CDC bootloader uploads now work. Turned out that AVRDUDE 5.10 could do the uploads but the newest version (6.3) can't. That means that we need to supply the correct version of AVRDUDE. Auto-build grabs the correct AVRDUDE image out of the Atom directory. Sometimes there's hangs at the end of the upload but the code is always there.
  3. Added more error handling when trying to move the AT90USB JSON file to the PlatformIO directory. If Auto-build can't verify the JSON is in the proper location it'll warn the user that they'll have to manually move the file and then gives the full source & destination paths.
  4. Added routine to calculate the percent FLASH & RAM used. This is only needed by the AT90USB boards when doing an upload. All the others situations already have this automatically done by PlatformIO. Sometimes this doesn't show up if the CDC upload hangs.
  5. At90USB boards will now only look for the upload port when doing an upload.
  6. Found another way a file path is presented during an error situation & added support for it.

This PR implements the last of the major features I like to use when debugging problems.

TO DO

  • Visual Studio support
  • Documentation

@thinkyhead thinkyhead merged commit 235facd into MarlinFirmware:bugfix-2.0.x May 26, 2018
@Bob-the-Kuhn Bob-the-Kuhn deleted the auto-build-avrdude branch May 26, 2018 12:53
thinkyhead added a commit that referenced this pull request Dec 16, 2024
Followup to #15347
Originally from #10849
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants