Skip to content

Merge Upload and Upload using programmer options #2073

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

Conversation

matthijskooijman
Copy link
Collaborator

This merges these two options, adding the normal "bootloader" upload method as just another entry in the "Programmer" selection menu. This cleans up the uploading code a bit and allows uploading using a programmer through the commandline now.

This adds a special value "internal" for the "programmer" preference.
When "programmer" is set to "internal", the regular bootloader upload is
used, as defined by boards.txt. For other values of the "programmer"
entry, the relevant info from programmers.txt is used.

This resolves arduino#2046.
@damellis
Copy link
Contributor

This means that if you select a programmer (e.g. to burn a boot loader), that programmer will also be used with the "Upload" command (until you select "boot loader" or similar from the programmer menu)?

That seems confusing: if a beginner selects a programmer to burn a bootloader, they won't expect that to break their uploads. Also, it's nice to be able to leave a programmer selected even as you do uploads via the boot loader. But really, I think the main issue is that I think this would confuse a lot of people, especially because it goes against behavior that's been in place for a long time.

I'd recommend against changing this in this way.

If what's wanted is a way to upload using programmer from the command line, I'd simply add that as a separate argument. (e.g. --upload-using-programmer instead of --upload, or --upload -programmer=)

@matthijskooijman matthijskooijman added Component: IDE The Arduino IDE Component: IDE user interface The Arduino IDE's user interface Version: 1.5.x feature request A request to make an enhancement (not a bug fix) and removed Component: IDE The Arduino IDE labels Sep 10, 2014
@ffissore ffissore self-assigned this May 12, 2015
@facchinm facchinm closed this Feb 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: IDE user interface The Arduino IDE's user interface feature request A request to make an enhancement (not a bug fix)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants