patacongo edited a comment on pull request #1346: URL: https://github.com/apache/incubator-nuttx/pull/1346#issuecomment-652567672
No.. no duplicate functionality. There is no functionality added by this PR that does not already exist in the OS. Perhaps you would would want your own private wrapper function to simplify the interfaces (like binfmt/binfmt_exec.c), but it is WRONG to force this pointless duplication onto the OS. Not only does this change duplicate existing functionality, but subverts a common OS internal interface to support special operations used on by Xiaomi and not for any other users. Subverting the OS for your own personal business development is not an acceptable practice. When you joined the Apache group, you agreed to support the OS for the good of the public with NO influence from your business. This is a bad technical change and it supports only Xaiomi's private interests and hence, has no business in a open operating system. If you were truly representing the interests of the OS and not your private interests you would understand this. You would understand perfectly if there were some other large business trying subvert the common interfaces for their personal interest. That is not an acceptable practice in an Apache project. ---------------------------------------------------------------- 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]
