Barto22 commented on PR #17849: URL: https://github.com/apache/nuttx/pull/17849#issuecomment-3776370765
Ok, what do you propose as the next steps? I’m finishing all tests locally and everything seems to be working. There will still be some fixes needed in NuttX and NuttX apps — for example, functions returning a value even though they’re declared void, and an issue with include guard naming in NuttX apps. How do you want to handle merging? Should I separate all build-related fixes into a dedicated PR's, and then submit another PR later with the updated Dockerfile? Also, in my other PR where I’m updating libcxx/libcxxabi, I’m reverting from version 21 to 19 so maybe it can build with the older toolchain. I’ll update that PR shortly and test it with the current NuttX Docker image. Let me know what you suggest, because updating the Dockerfile toolchain requires some code adjustments as well. -- 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]
