-
Notifications
You must be signed in to change notification settings - Fork 13.3k
Debug tracing menu items not shown for NodeMCU board #1969
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
Comments
Likewise for the Wemos boards |
This just needs the entries copied in boards.txt;
I've wondered, given the repetition if it might be worth having a script to
generate the file, which would avoid having to edit all the (other than key
name) identical bits for any and every change.
I've duplicated it for the NodeMCU (which I've been using) so it is just as
simple as copying and pasting the lines (updating the key names).
…On Tue, Jan 31, 2017 at 6:50 AM, tkerby ***@***.***> wrote:
Likewise for the Wemos boards
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#1969 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAN_A0hGZMYiVreetTUycLu2faT6JkG_ks5rXiLfgaJpZM4IOdtA>
.
|
Generating boards.txt during build phase (i.e. building boards manager package or bootstrapping a version cloned from git) is the way to go, I think. This also dovetails with your suggestion to keep all flash geometry info in boards.txt. Maybe we can keep it in the linker scripts, and generate boards.txt automatically? Or have another file with all the flash sizes data and generate boards.txt and linker scripts from it. |
I had wondered about that too, instead of complicating the command lines a
package-build step which would automate the lot including the various menu
options. (Though ultimately a more hierarchical config system for the IDE
would be nice)
If that were the way forward, what do we write such a script in? Are there
any existing similar scripts? I'd rather not add too many arbitrary
dependencies :)
…On Tue, Jan 31, 2017 at 8:37 PM, Ivan Grokhotkov ***@***.***> wrote:
Generating boards.txt during build phase (i.e. building boards manager
package or bootstrapping a version cloned from git) is the way to go, I
think. This also dovetails with your suggestion to keep all flash geometry
info in boards.txt. Maybe we can keep it in the linker scripts, and
generate boards.txt automatically? Or have another file with all the flash
sizes data and generate boards.txt and linker scripts from it.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#1969 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAN_Aw7F4oOGq7SZ3dT8zWaGdjB2FDQeks5rXuSggaJpZM4IOdtA>
.
|
We already employ some amount of Python (tools/get.py, and the unit test runner), so writing this in Python wouldn't add more dependencies. Another possibile option is to define all the input data in XML and use a couple of XSLT scripts to generate boards.txt and linker script files. I can't immediately tell if this will work well as I don't remember if it is possible to do the equivalent of nested loops in XSLT. |
I'd vote for Python (would have been my first choice :) so shall I organise
such a script in that in preference to adjusting linker command lines?
…On Tue, Jan 31, 2017 at 8:57 PM, Ivan Grokhotkov ***@***.***> wrote:
We already employ some amount of Python (tools/get.py, and the unit test
runner), so writing this in Python wouldn't add more dependencies.
Another possibile option is to define all the input data in XML and use a
couple of XSLT scripts to generate boards.txt and linker script files. I
can't immediately tell if this will work well as I don't remember if it is
possible to do the equivalent of nested loops in XSLT.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#1969 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAN_Ax34ncE2bmpB35sE0dCsKBUlK_t5ks5rXul1gaJpZM4IOdtA>
.
|
This script would allow us to handle both the linker scripts issue and this one, so I would vote for writing the script. In the past, we had some bugs due to incomplete/inconsistent updates to boards.txt, and such a script could help as well. It will also make adding new options and boards easier. |
The Menus are undergoing a full rework, which is tracked in #3582 . Closing. |
If NodeMCU board is selected in the IDE, no debug menus (for configuring debug serial output) are shown.
Hardware
Hardware: NodeMCP v1.0
Core Version: latest esp8266 from github?
Description
The required menu defs are just missing from the NodeMCU section of boards.txt. It was possible to get these added with some copy and pasting.
Settings in IDE
Module: NodeMCU v1.0
Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.
The text was updated successfully, but these errors were encountered: