casaroli commented on PR #19798:
URL: https://github.com/apache/nuttx/pull/19798#issuecomment-5290873141

   > > ## Summary
   > > This gives the ESP32-S3 a kernel build: `fork()`, a per-process address 
environment, and a privilege boundary between the kernel and a user process.
   > > It replaces #19795, #19796 and #19797, which are the same work in three 
parts.
   > 
   > Why not let reviewer continue on #19795, #19796 and #19797? it's better to 
split your huge change into the small and self-contained pr, otherwise any 
concern will block your whole change to merge.
   
   I agree to split, however, i'd like to keep a PR with all the changes so 
they can test the final intended result.
   
   I updated the PR body to not mention that this pr replaces the others


-- 
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]

Reply via email to