Re: git: eca005d8531f - main - zlib: silence K&R warns

From: Jessica Clarke <jrtc27_at_freebsd.org>
Date: Sun, 12 Feb 2023 21:36:54 UTC
On 12 Feb 2023, at 21:23, Mateusz Guzik <mjguzik@gmail.com> wrote:
> 
> On 2/12/23, Jessica Clarke <jrtc27@freebsd.org> wrote:
>> On 12 Feb 2023, at 21:13, Mateusz Guzik <mjg@FreeBSD.org> wrote:
>>> 
>>> The branch main has been updated by mjg:
>>> 
>>> URL:
>>> https://cgit.FreeBSD.org/src/commit/?id=eca005d8531fd153f46f941f3101e73cdf181a0b
>>> 
>>> commit eca005d8531fd153f46f941f3101e73cdf181a0b
>>> Author:     Mateusz Guzik <mjg@FreeBSD.org>
>>> AuthorDate: 2023-02-12 21:12:34 +0000
>>> Commit:     Mateusz Guzik <mjg@FreeBSD.org>
>>> CommitDate: 2023-02-12 21:12:34 +0000
>>> 
>>>   zlib: silence K&R warns
>>> 
>>>   Sponsored by:   Rubicon Communications, LLC ("Netgate")
>>> ---
>>> sys/conf/files            | 18 +++++++++++-------
>>> sys/modules/zlib/Makefile |  2 ++
>>> 2 files changed, 13 insertions(+), 7 deletions(-)
>>> 
>>> diff --git a/sys/conf/files b/sys/conf/files
>>> index 9171b434a7bf..53f90f7afc22 100644
>>> --- a/sys/conf/files
>>> +++ b/sys/conf/files
>>> @@ -4099,28 +4099,32 @@ libkern/strtouq.c		standard
>>> libkern/strvalid.c		standard
>>> libkern/timingsafe_bcmp.c	standard
>>> contrib/zlib/adler32.c		optional crypto | geom_uzip | \
>>> -	mxge | ddb_ctf | gzio | zfs | zlib
>>> +	mxge | ddb_ctf | gzio | zfs | zlib \
>>> +	compile-with "${NORMAL_C} -Wno-deprecated-non-prototype"
>> 
>> This doesn’t exist pre-Clang 15.
>> 
> 
> this is main-only and it already has llvm15. the build requirement is
> top of the branch, so I don't see a problem.
> 
> if you ran into trouble cross-compiling, you should probably update
> that env instead of expecting working around the warn in the tree for
> older compilers

That is not our policy. Please fix this.

Jess

>> Jess
>> 
>>> contrib/zlib/compress.c		optional crypto | geom_uzip | \
>>> 	mxge | ddb_ctf | gzio | zfs | zlib \
>>> -	compile-with "${NORMAL_C} -Wno-cast-qual"
>>> +	compile-with "${NORMAL_C} -Wno-cast-qual -Wno-deprecated-non-prototype"
>>> contrib/zlib/crc32.c		optional crypto | geom_uzip | \
>>> 	mxge | ddb_ctf | gzio | zfs | zlib \
>>> 	compile-with "${NORMAL_C} ${NO_WSTRICT_PROTOTYPES}"
>>> contrib/zlib/deflate.c		optional crypto | geom_uzip | \
>>> 	mxge | ddb_ctf | gzio | zfs | zlib \
>>> -	compile-with "${NORMAL_C} -Wno-cast-qual"
>>> +	compile-with "${NORMAL_C} -Wno-cast-qual -Wno-deprecated-non-prototype"
>>> contrib/zlib/inffast.c		optional crypto | geom_uzip | \
>>> -	mxge | ddb_ctf | gzio | zfs | zlib
>>> +	mxge | ddb_ctf | gzio | zfs | zlib \
>>> +	compile-with "${NORMAL_C} -Wno-deprecated-non-prototype"
>>> contrib/zlib/inflate.c		optional crypto | geom_uzip | \
>>> -	mxge | ddb_ctf | gzio | zfs | zlib
>>> +	mxge | ddb_ctf | gzio | zfs | zlib \
>>> +	compile-with "${NORMAL_C} -Wno-deprecated-non-prototype"
>>> contrib/zlib/inftrees.c		optional crypto | geom_uzip | \
>>> -	mxge | ddb_ctf | gzio | zfs | zlib
>>> +	mxge | ddb_ctf | gzio | zfs | zlib \
>>> +	compile-with "${NORMAL_C} -Wno-deprecated-non-prototype"
>>> contrib/zlib/trees.c		optional crypto | geom_uzip | \
>>> 	mxge | ddb_ctf | gzio | zfs | zlib \
>>> 	compile-with "${NORMAL_C} ${NO_WSTRICT_PROTOTYPES}"
>>> contrib/zlib/uncompr.c		optional crypto | geom_uzip | \
>>> 	mxge | ddb_ctf | gzio | zfs | zlib \
>>> -	compile-with "${NORMAL_C} -Wno-cast-qual"
>>> +	compile-with "${NORMAL_C} -Wno-cast-qual -Wno-deprecated-non-prototype"
>>> contrib/zlib/zutil.c		optional crypto | geom_uzip | \
>>> 	mxge | ddb_ctf | gzio | zfs | zlib \
>>> 	compile-with "${NORMAL_C} ${NO_WSTRICT_PROTOTYPES}"
>>> diff --git a/sys/modules/zlib/Makefile b/sys/modules/zlib/Makefile
>>> index 6391255e512b..178d3a3f7ab7 100644
>>> --- a/sys/modules/zlib/Makefile
>>> +++ b/sys/modules/zlib/Makefile
>>> @@ -25,4 +25,6 @@ CWARNFLAGS.crc32.c+=${NO_WSTRICT_PROTOTYPES} #
>>> crc32.c:586
>>> CWARNFLAGS.trees.c+=${NO_WSTRICT_PROTOTYPES} # trees.c:232
>>> CWARNFLAGS.zutil.c+=${NO_WSTRICT_PROTOTYPES} # zutil.c:27,32
>>> 
>>> +CWARNFLAGS+=-Wno-deprecated-non-prototype
>>> +
>>> .include <bsd.kmod.mk>
>> 
>> 
> 
> 
> -- 
> Mateusz Guzik <mjguzik gmail.com>