Re: git: 57a30ab234a9 - main - net/tcpview: Fix build with LLVM15

From: Cy Schubert <Cy.Schubert_at_cschubert.com>
Date: Sun, 12 Feb 2023 03:38:39 UTC
In message <ilg8-nsea-wny@FreeBSD.org>, Jan Beich writes:
> Cy Schubert <cy@FreeBSD.org> writes:
>
> > The branch main has been updated by cy:
> >
> > URL: https://cgit.FreeBSD.org/ports/commit/?id=57a30ab234a9771eb5a4f50a54f8
> aaba7cd62a91
> >
> > commit 57a30ab234a9771eb5a4f50a54f8aaba7cd62a91
> > Author:     Cy Schubert <cy@FreeBSD.org>
> > AuthorDate: 2023-02-11 13:54:07 +0000
> > Commit:     Cy Schubert <cy@FreeBSD.org>
> > CommitDate: 2023-02-11 13:54:07 +0000
> >
> >     net/tcpview: Fix build with LLVM15
> > ---
> >  net/tcpview/Makefile | 1 +
> >  1 file changed, 1 insertion(+)
> >
>
> Please, document errors shoved under the carpet in commit messages.

Sure.

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 returning 'void *' from a function with result type 'int' [-Wint-conversion]
            return NULL;
                   ^~~~
/usr/include/sys/_null.h:34:14: note: expanded from macro 'NULL'
#define NULL    ((void *)0)
                ^~~~~~~~~~~

The complete build log can be found here:

https://pkg-status.freebsd.org/beefy17/data/main-i386-default/p58633340e9ed_s5636590214/logs/tcpview-1.0_7.log

and here:

https://pkg-status.freebsd.org/beefy18/data/main-amd64-default/p58633340e9ed_s5636590214/logs/tcpview-1.0_7.log

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.


-- 
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