-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
[Build] Add short compressed build description (#3262) #3263
base: mega
Are you sure you want to change the base?
Conversation
I think this would also need a Python counterpart, to define the filename before the compile is started? |
Wouldn't it be more natural to first list the 'P' plugins bitmap, then the 'T' controllers bitmap and then the 'N' notifications bitmap? (to be implemented). I think most users will first look at the plugins before looking at the controllers, but that may just be me. |
In size it wouldn't matter and in decoding probably also makes no difference. Maybe a JS piece of code somewhere online where a user can decode the string into readable form. |
Been chewing a bit in the back of my mind for a couple of weeks now, and it feels more and more like a way to go forward. It may need a way to include the 'other' compile-time flags like |
See letscontrolit#3262 Outputs `Tffff:7:P17:2400:22:(3):10:` Equivalent to: ``` "CONTROLLER_SET_ALL", "NOTIFIER_SET_NONE", "PLUGIN_SET_ONLY_SWITCH", "USES_P001", # Switch "USES_P002", # ADC "USES_P004", # Dallas DS18b20 "USES_P100", # Pulse Counter - DS2423 "USES_C016", # Cache Controller "USES_C018", # TTN/RN2483 "USES_C015", # TTN/RN2483 ``` Still missing Notifier, CPU, flash size. [Build] Fix bit order in build description Fix build issues after merge
07f2539
to
abd5757
Compare
Nee to add a bitmap too for all the |
These are the (global) FEATURE_(flags) I found after converting all non-plugin, non-controller, non-Notifier USE_/USES_ settings:
|
See #3262
Outputs
Tffff:e000:Pe800:24:4400:(3):800:
Equivalent to:
Still missing Notifier, CPU, flash size.