xiaoxiang781216 commented on PR #9400: URL: https://github.com/apache/nuttx/pull/9400#issuecomment-1565598185
> > > Another thing that I would like to know is: Would you guys prefer to maintain only one file and merge the current `version.h` into the new `info.h` ? > > > > > > since it's only used by nxdiag, why not use a python script like https://github.com/apache/nuttx-apps/blob/master/tools/host_sysinfo.py to generate this information? > > @xiaoxiang781216 since this is a simple script and it could become a de-facto solution for all other boards, I think it is better keep it in C instead of python. We don't want to depend on python for basic things, it could prevent us to compile NuttX inside a machine running NuttX in the future (case we get some compiler like tcc running on NuttX). It's fine to develop the tool by C or Python. My major concern is that it isn't good to generate it unconditionaly since this information is only used by nxdiag. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
