[GitHub] [incubator-nuttx] hartmannathan edited a comment on pull request #5070: libc/semaphore:sem_init change defult protocol

[GitBox <gi...@apache.org>]

hartmannathan edited a comment on pull request #5070:
URL: https://github.com/apache/incubator-nuttx/pull/5070#issuecomment-1000898688


   **This will also break many things for downstream projects using NuttX.**
   
   We need to make sure the wider community notices this proposed change so it can be reviewed properly. I will post on the mailing list, but there are other venues, so to everyone frequenting them, please help draw attention to this there, too.
   
   To be clear, I am in favor of standards-compliance, provided this is in line with NuttX's [Inviolables](https://github.com/hartmannathan/incubator-nuttx/blob/master/INVIOLABLES.md):
   - If it is standard POSIX behavior to do as this change proposes, then we should do so, but only after making sure it is reviewed properly.
   - If this is Linux behavior and is in contradiction to POSIX, then we should **not** do it.
   
   Also, if this change is implemented, we need to increment the major version number. It will have to be NuttX 11.x.


-- 
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: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org