tmedicci commented on PR #15304: URL: https://github.com/apache/nuttx/pull/15304#issuecomment-2569178461
> the problem is that it's not good to put the arch/chip specific code in a common script. My suggestion is to decouple the `host_sysinfo.py` from chip-specific. Let's split Espressif-related functions to `tools/espressif/chip_info.py` (and similarly for other vendors) and, based on `.config` file, we can import it to `host_sysino.py` internally. It'd be like a plugin that imports chip-specific information. Instead of using the `--espressif` argument, we can use `--enable-target-info` to enable/disable such set of chip-specific information. > I would suggest renaming `debug_info` target to `sysinfo` I agree! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@nuttx.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
