nodemcu-firmware icon indicating copy to clipboard operation
nodemcu-firmware copied to clipboard

Feature request: Machine-readable description of build config options

Open mk-pmb opened this issue 4 years ago • 11 comments

Missing feature

Let's make the comments in app/include/user_*.h more machine-readable:

  • macro name
  • default value
  • category
  • importance level or sth.
  • short description
  • (optional) detailed description
  • Possible values
    • (optional) short description
    • (optional) detailed description

In case we already have such info, let's put a hint where to find it on top of the header files. And establish a mechanism for keeping them in sync.

Justification

I'd like to make a config wizard. That wizard, at a minimum, should be able to determine whether it's understanding of available options is still compatible with the currently checked-out version of the firmware. Would be nice if it could also fall back to the maintainer's help texts if it cannot find localized/optimized help texts from a user-preferred source.

Workarounds

If it turns out we don't have such info, I might have to bake my own.

mk-pmb avatar Dec 31 '19 16:12 mk-pmb

Progess: I compiled short descriptions for all documented modules. Preparations for categorization.

mk-pmb avatar Jan 01 '20 04:01 mk-pmb

That sounds like a good idea. There is also a possible related issue #386 . Please also note that in the near future configuration will be changed for esp8266 to work as in esp32, So editing the user_config will be history then.

HHHartmann avatar Jan 01 '20 17:01 HHHartmann

Can you/someone estimate how far away that future is?

mk-pmb avatar Jan 01 '20 17:01 mk-pmb

in the near future configuration will be changed for esp8266 to work as in esp32

Cool, who is working on that? It's the first time I hear this. I was also going to propose to @mk-pmb that before he reinvents the wheel he should look into what ESP-IDF does with Kconfig.

marcelstoer avatar Jan 01 '20 22:01 marcelstoer

I tried to find where I read it but can't. Seems that I didn't read close enough. It is between https://github.com/nodemcu/nodemcu-firmware/issues/1657#issuecomment-441338203 and a testrun to integrate and build the two variants in one repo by someone (don't remember who) that I got that impression. So sorry for the noise.

HHHartmann avatar Jan 02 '20 15:01 HHHartmann

in the near future configuration will be changed for esp8266 to work as in esp32

Cool, who is working on that?

This came up in some discussion between Johny and me, but ATM @jmattsson has even less bandwidth on NodeMCU than I do.

TerryE avatar Jan 31 '20 12:01 TerryE

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Jan 31 '21 01:01 stale[bot]

Any news on this?

mk-pmb avatar Jan 31 '21 06:01 mk-pmb

It's still on my todo list, just like last year >.<

On Sun, 31 Jan 2021 at 17:18, M.K. [email protected] wrote:

Any news on this?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/nodemcu/nodemcu-firmware/issues/2989#issuecomment-770334075, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAGUJZ4LWCFTEMJ6K2BFRXDS4TYZRANCNFSM4KBWVZUA .

jmattsson avatar Jan 31 '21 06:01 jmattsson

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Apr 16 '22 07:04 stale[bot]

Is anyone still working on this, or has it on their todo?

mk-pmb avatar Apr 16 '22 18:04 mk-pmb