Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 14 Feb 2023 20:39:16 -0800
From:      Cy Schubert <Cy.Schubert@cschubert.com>
To:        Alexey Dokuchaev <danfe@freebsd.org>
Cc:        Cy Schubert <Cy.Schubert@cschubert.com>, Jan Beich <jbeich@freebsd.org>, Cy Schubert <cy@freebsd.org>, ports-committers@freebsd.org, dev-commits-ports-all@freebsd.org, dev-commits-ports-main@freebsd.org
Subject:   Re: git: 57a30ab234a9 - main - net/tcpview: Fix build with LLVM15
Message-ID:  <20230215043916.4A72A6CE@slippy.cwsent.com>
In-Reply-To: <Y%2Bxe7dWF724URxsK@FreeBSD.org>
References:  <202302111441.31BEf32w000463@gitrepo.freebsd.org>  <ilg8-nsea-wny@FreeBSD.org> <20230212033839.6259B28E@slippy.cwsent.com>  <Y%2Bxe7dWF724URxsK@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
In message <Y+xe7dWF724URxsK@FreeBSD.org>, Alexey Dokuchaev writes:
> On Sat, Feb 11, 2023 at 07:38:39PM -0800, Cy Schubert wrote:
> > ...
> > The exact error, due to -Wint-conversion now being the default in LLVM15 --
> > https://reviews.llvm.org/D129881.
> > 
> > detail-domain.c:449:13: error: incompatible pointer to integer conversion r
> eturning 'void *' from a function with result type 'int' [-Wint-conversion]
> >             return NULL;
> >                    ^~~~
>
> So why not fix the broken code rather than suppress the useful warning?
> New strict Clang already revealed several real bugs in various ports,
> thus I encourage everyone not to silence the compiler, but examine and
> address the actual problem with the code.

This is a medium-term goal ATM. A number of nasty kerberos bugs have taken 
precidence.

>
> > Previously there would have been an exp-run prior to a new LLVM import
> > allowing for better planning and staging than the current react mode.
>
> There was an exp-run (PR 265425), but apparently dim@ concentrated on
> trying to fix relatively high-profile ports which unblock N [other]
> ports, so many affected leaf ports got under his radar.

There's a lot of stuff going on here. I may have missed that.

>
> ./danfe


-- 
Cheers,
Cy Schubert <Cy.Schubert@cschubert.com>
FreeBSD UNIX:  <cy@FreeBSD.org>   Web:  https://FreeBSD.org
NTP:           <cy@nwtime.org>    Web:  https://nwtime.org

			e^(i*pi)+1=0





Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20230215043916.4A72A6CE>