Re: some warnings in prep for LLVM 13

2021-10-25 Thread Jeremie Courreges-Anglas
On Mon, Oct 25 2021, Mark Kettenis wrote: >> From: Jeremie Courreges-Anglas >> Date: Mon, 25 Oct 2021 14:55:16 +0100 >> >> On Mon, Oct 25 2021, Mark Kettenis wrote: >> >> Date: Mon, 25 Oct 2021 12:37:41 +0200 (CEST) >> >> From: Mark Kettenis >> >> >> >> > Date: Mon, 25 Oct 2021 12:01:11

Re: some warnings in prep for LLVM 13

2021-10-25 Thread Mark Kettenis
> From: Jeremie Courreges-Anglas > Date: Mon, 25 Oct 2021 14:55:16 +0100 > > On Mon, Oct 25 2021, Mark Kettenis wrote: > >> Date: Mon, 25 Oct 2021 12:37:41 +0200 (CEST) > >> From: Mark Kettenis > >> > >> > Date: Mon, 25 Oct 2021 12:01:11 +0200 > >> > From: Patrick Wildt > >> > > >> > Hi, >

Re: some warnings in prep for LLVM 13

2021-10-25 Thread Jeremie Courreges-Anglas
On Mon, Oct 25 2021, Mark Kettenis wrote: >> Date: Mon, 25 Oct 2021 12:37:41 +0200 (CEST) >> From: Mark Kettenis >> >> > Date: Mon, 25 Oct 2021 12:01:11 +0200 >> > From: Patrick Wildt >> > >> > Hi, >> > >> > mortimer@ has this diff in his tree for LLVM 13. I actually haven't >> > tried to

Re: some warnings in prep for LLVM 13

2021-10-25 Thread Mark Kettenis
> Date: Mon, 25 Oct 2021 12:37:41 +0200 (CEST) > From: Mark Kettenis > > > Date: Mon, 25 Oct 2021 12:01:11 +0200 > > From: Patrick Wildt > > > > Hi, > > > > mortimer@ has this diff in his tree for LLVM 13. I actually haven't > > tried to see if it works fine with LLVM 11, but I feel it needs

Re: some warnings in prep for LLVM 13

2021-10-25 Thread Theo Buehler
On Mon, Oct 25, 2021 at 01:46:51PM +0200, Sebastien Marie wrote: > I have a working llvm13 here for building zig 0.9.0-dev. > > /usr/src/usr.bin/openssl/s_client.c:897:16: error: variable 'pbuf_off' set > but not used [-Werror,-Wunused-but-set-variable] > int pbuf_len, pbuf_off; >

Re: some warnings in prep for LLVM 13

2021-10-25 Thread Sebastien Marie
On Mon, Oct 25, 2021 at 12:35:03PM +0100, Jeremie Courreges-Anglas wrote: > On Mon, Oct 25 2021, Theo Buehler wrote: > >> index 664a5200037..e33763e7420 100644 > >> --- a/usr.bin/openssl/Makefile > >> +++ b/usr.bin/openssl/Makefile > >> @@ -17,6 +17,7 @@ CFLAGS+= -Wuninitialized > >> CFLAGS+=

Re: some warnings in prep for LLVM 13

2021-10-25 Thread Jeremie Courreges-Anglas
On Mon, Oct 25 2021, Theo Buehler wrote: >> index 664a5200037..e33763e7420 100644 >> --- a/usr.bin/openssl/Makefile >> +++ b/usr.bin/openssl/Makefile >> @@ -17,6 +17,7 @@ CFLAGS+= -Wuninitialized >> CFLAGS+= -Wunused >> .if ${COMPILER_VERSION:L} == "clang" >> CFLAGS+= -Werror >> +CFLAGS+=

Re: some warnings in prep for LLVM 13

2021-10-25 Thread Mark Kettenis
> Date: Mon, 25 Oct 2021 12:01:11 +0200 > From: Patrick Wildt > > Hi, > > mortimer@ has this diff in his tree for LLVM 13. I actually haven't > tried to see if it works fine with LLVM 11, but I feel it needs to be > sent out and not just be blindly committed. > > If someone wants to take care

Re: some warnings in prep for LLVM 13

2021-10-25 Thread Theo Buehler
> index 664a5200037..e33763e7420 100644 > --- a/usr.bin/openssl/Makefile > +++ b/usr.bin/openssl/Makefile > @@ -17,6 +17,7 @@ CFLAGS+= -Wuninitialized > CFLAGS+= -Wunused > .if ${COMPILER_VERSION:L} == "clang" > CFLAGS+= -Werror > +CFLAGS+= -Wno-unused-but-set-variable This will break the

some warnings in prep for LLVM 13

2021-10-25 Thread Patrick Wildt
Hi, mortimer@ has this diff in his tree for LLVM 13. I actually haven't tried to see if it works fine with LLVM 11, but I feel it needs to be sent out and not just be blindly committed. If someone wants to take care of this, it would be nice, so I can take care of the remaining parts of sending