Hi,

it's been 72 hours since the vote start with 7 +1 votes and no 0 or -1
votes. I think we can close the vote now and I'll mark the pull request
as ready. Thanks.

Michal

On 4/29/25 08:37, Michal Lenc wrote:
> Hi all,
>
> I've submitted pull request that changes setlogmask function behavior to
> the one expected by POSIX standard
> <https://github.com/apache/nuttx/pull/16280>. The description of the
> change is provided in the mailing list, to summarize it, our
> implementation uses zero argument to set logging mask to zero and thus
> disable the logging at all. However, POSIX clearly states that passing
> zero argument should not modify the current logging mask, but just
> return the old one. We should change the behavior to comply with POSIX
> standards here.
>
> As Tomek suggested, we should vote here first, because this is a
> potential breaking change for existing applications. So let's vote
> for/against the change.
>
> It's also worth noting that POSIX probably (or at least I haven't found
> a way) doesn't allow to disable all logging at runtime, just to set the
> mask to the highest priority (lowest amount of messages, just emergency
> logs) with LOG_EMERG (or any other priority you want). Nevertheless, I
> consider the POSIX (and Linux) compatibility more important in this case
> and vote for the change.
>
> Thanks,
>
> Michal
>

Reply via email to