git: 451ac17a5030 - main - stans: Narrow the scope of includes and other flags
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Fri, 08 Jul 2022 17:50:01 UTC
The branch main has been updated by imp: URL: https://cgit.FreeBSD.org/src/commit/?id=451ac17a50303fec79797e900c7a37914adf9b37 commit 451ac17a50303fec79797e900c7a37914adf9b37 Author: Warner Losh <imp@FreeBSD.org> AuthorDate: 2022-07-08 15:42:34 +0000 Commit: Warner Losh <imp@FreeBSD.org> CommitDate: 2022-07-08 17:47:36 +0000 stans: Narrow the scope of includes and other flags CFLAGS+= here affects *ALL* libsa files being built. However, this is only needed for zfs.c, so define it only for this. Also, use the defines from defs.mk. Move all the zfs.c include hacks together. Also, move the -Wformat -Wall warnings that were added to CFLAGS+= to the individual files instead for the same reason. Sponsored by: Netflix --- stand/libsa/zfs/Makefile.inc | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/stand/libsa/zfs/Makefile.inc b/stand/libsa/zfs/Makefile.inc index 841b1b5c327c..f847af8949ff 100644 --- a/stand/libsa/zfs/Makefile.inc +++ b/stand/libsa/zfs/Makefile.inc @@ -30,6 +30,7 @@ ZFS_EARLY= -I${ZFSOSINC} \ .for i in ${ZFS_SRC} ${ZSTD_SRC} CFLAGS.$i+= -include ${ZFSOSINC}/spl/sys/ccompile.h +CFLAGS.$i+= -Wformat -Wall .endfor CFLAGS_EARLY.list.c+= ${ZFS_EARLY} @@ -39,7 +40,9 @@ CFLAGS_EARLY.zstd_shim.c+= ${ZFS_EARLY} # the zfs code that need to be unified. CFLAGS.nvlist.c+= -I${ZFSOSINC}/spl CFLAGS.zfs.c+= -I${ZFSOSINC}/spl \ - -I${SRCTOP}/sys/cddl/contrib/opensolaris/common/lz4 + -I${SRCTOP}/sys/cddl/contrib/opensolaris/common/lz4 \ + -I${ZFSOSINC}/zfs \ + -I${OZFS}/include # # ZSTD coding style has some issues, so suppress clang's warnings. Also, zstd's @@ -53,9 +56,3 @@ CFLAGS.zstd_shim.c+= -DIN_BASE -DIN_LIBSA -I${OZFS}/include # Do not unroll skein loops, reduce code size CFLAGS.skein_block.c+= -DSKEIN_LOOP=111 - -CFLAGS+= -I${SYSDIR}/contrib/openzfs/include -CFLAGS+= -I${SYSDIR}/contrib/openzfs/include/os/freebsd/zfs -CFLAGS.zfs.c+= -I${SYSDIR}/cddl/contrib/opensolaris/common/lz4 - -CFLAGS+= -Wformat -Wall