raiden00pl commented on PR #17352: URL: https://github.com/apache/nuttx/pull/17352#issuecomment-3556816523
@anchao I understand the benefits of disabling signals and I like these, but as I mentioned, it's not POSIX-compliant. PSE51 also requires signal support, even if they make no practical sense in this case. On the one hand, we want to remove `time_t` as 32-bit (https://github.com/apache/nuttx/pull/14460), which is POSIX-complaint, and on the other, we want to disable signals which is POSIX-incompatible. I see some inconsistency here. It would be good to establish some rules about when we can accept breaking POSIX and when we can't. I think we have a condition for this in `INVIOLABLES.md`, but it is not precise: https://github.com/apache/nuttx/blob/fd70e5f947cb38f83503cc57b897e9cfb8368d9f/INVIOLABLES.md?plain=1#L29-L30 In this case voting on the mailing list would be a good idea. BTW, do you have any comparisons between footprint and performers that could be presented as an argument for this change? -- 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]
