v01d commented on pull request #3470: URL: https://github.com/apache/incubator-nuttx/pull/3470#issuecomment-815210957
> > There's already mention of this in the doc I migrated. Still, if we already provide that .cfg I don't see much point in explaining how to recreate it. > > Actually, I also wanted to remove those .cfg. The important part is already in those distributed with OpenOCD, the only thing that we need to add is the RTOS part. Sounds good. That can be done in a separate PR while updating the docs to only mention the changes to make to the .cfg. > > > The instruction to download the toolchain is already there (you can actually already see it on the doc website). Please not that this is at the ESP32 level (not board level). My policy in this PR was to mostly migrate things and not remove big things. I would suggest to do that in a second PR as I mentioned. > > Right, sorry I missed it. This PR is fine as it is, I'm not suggesting to add/remove all those things I was commenting about. Ok. We can iterate this later on anyway. -- 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. For queries about this service, please contact Infrastructure at: [email protected]
