Source-Changes-D archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/sys/rump



On 09.03.2020 15:31, Taylor R Campbell wrote:
>> Module Name:    src
>> Committed By:   kamil
>> Date:           Mon Mar  9 00:03:00 UTC 2020
>>
>> Modified Files:
>>         src/sys/rump: Makefile.rump
>>
>> Log Message:
>> Build RUMP with -fno-delete-null-pointer-checks on all compilers
> 
> I asked you to hold off on making this change, ten hours before you
> proceeded to make it:
> 
> https://mail-index.NetBSD.org/tech-kern/2020/03/08/msg026125.html
> 
> It remains unclear whether the change is actually necessary.
> 

It is necessary. The tool detects real UB and the tool is not broken.

I went that far that I summoned people from compilers and the C + C++
standard to confirm this.

> Please back this out until the discussion has reached a conclusion,
> and please don't unilaterally move ahead to make changes that are
> still under active discussion -- especially when someone in the
> discussion you initiated has specifically asked you to wait.
> 
> If there is a dispute that has deadlocked in public discussion and
> requires a resolution, then you can present a case to core and ask for
> a ruling.  But right now the discussion is not deadlocked and nobody
> has asked for core to step in to resolve any dispute in order to make
> progress.
> 

I will revert it for the time being on this demand.

Attachment: signature.asc
Description: OpenPGP digital signature



Home | Main Index | Thread Index | Old Index