patacongo commented on pull request #4271: URL: https://github.com/apache/incubator-nuttx/pull/4271#issuecomment-891837005
That rule has been around for many years. The reason is for "marketing." In several instances, people evaluating NuttX built a configuration they were interested in to see the size and performance of the OS. It just so happened that DEBUG options were enabled in those configurations. There is no way for anyone to know. The result was that they concluded the the OS was too big, too slow, and too inefficient. Having DEBUG enabled in defconfigs has actually cost us users! CONFIG_EXPERIMENTAL does not help in that scenario. I think if you want debug enabled, then the defconfig directory should be have a -debug suffix. For example, rndis would be a production RNDIS configuration; rndis-debug would be a configuration with debug features enabled. I think that would meet both needs. A few years back, we went through and removed DEBUG options from all configurations. It sounds like more have crept back in over the years. Those defconfig files should all have the DEBUG options removed. -- 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]
