Skip to content

Consolidate the programmer options to be more logical and flexible #2046

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

Open
tchilton opened this issue May 3, 2014 · 6 comments
Open

Consolidate the programmer options to be more logical and flexible #2046

tchilton opened this issue May 3, 2014 · 6 comments
Labels
feature request A request to make an enhancement (not a bug fix)

Comments

@tchilton
Copy link

tchilton commented May 3, 2014

The configuration of programmers and use of them is inconsistent in the UI and config files 1.5.6. For example :

Preferences file upload.using option - to set bootloader vs others. Why is bootloader not just another programmer type under tools, programmer ?, then preference could be removed and consistency provided

This would then allow the upload and upload using programmer to be consolidated to just upload - with the type set by the tools programmer option, which would also allow the upload button on the UI to work across both types of programming.

There are also two bugs in the programing tool executable selection from boards.txt. If you set another programmer other than avrdude, even when the new executable is in the same folder as avrdudem it fails to spawn that process.
Similarly, if you use tools burn bootloader in the same condition, it claims the file is not present when it is.

@matthijskooijman
Copy link
Collaborator

Ha! I was thinking exactly the same thing a few days ago, so +1 from me :-)

As for the two issues you mention, perhaps you should create a separate github issue for them (perhaps even two?) and include a bit more detail. It sounds like you're modifying boards.txt and platform.txt with custom tools, so best include the actual changes you made in your bug report so we can figure out what's going on.

@tchilton
Copy link
Author

tchilton commented May 4, 2014

Following a pointer from Matthijs on issue #2047, the two bugs turned out not to be so. it was my understanding of the boards.txt / programmer.txt - I had missed the linkage through platform.txt..

The rest re the UI side of things remains an issue though.

matthijskooijman added a commit to matthijskooijman/Arduino that referenced this issue May 13, 2014
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.
@matthijskooijman
Copy link
Collaborator

@tchilton After implementing your suggestion, some discussion ensued on the developer list. Perhaps you care to chime in? https://groups.google.com/a/arduino.cc/d/msg/developers/CPnr4mPle7Q/k1j-EnkSSZ8J

@tchilton
Copy link
Author

No problem, I've submitted my comments on the Google Groups discussion.
(Why are there two discussion places ??)

@matthijskooijman
Copy link
Collaborator

(Why are there two discussion places ??)

You mean the forum and the list, or?

@tchilton
Copy link
Author

It was a poor joke relating to the two programming options at the top of this thread, now we have two forums to discuss it on .. ;-)

@ffissore ffissore self-assigned this Jul 1, 2015
@per1234 per1234 added the feature request A request to make an enhancement (not a bug fix) label Jul 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request A request to make an enhancement (not a bug fix)
Projects
None yet
Development

No branches or pull requests

4 participants