Differential D4515 Diff 11159 sys/boot/efi/boot1/Makefile

Changeset View

Standalone View

sys/boot/efi/boot1/Makefile

Context not available.

	MAN=	MAN=

		CWARNFLAGS?= -Wall -Wredundant-decls -Wnested-externs -Wstrict-prototypes \
		-Wmissing-prototypes -Wpointer-arith -Winline -Wcast-qual \
		-Wundef -Wno-pointer-sign -Wmissing-include-dirs \
		-fdiagnostics-show-option -Wno-unknown-pragmas
		emasteUnsubmitted Done Inline Actions Can you explain why individual warning flags are turned on here? emaste: Can you explain why individual warning flags are turned on here?
		smhAuthorUnsubmitted Done Inline Actions All the flags from kern.mk smh: All the flags from kern.mk
		emasteUnsubmitted Done Inline Actions is there not a WARNS= we can use somehow that gets (most of) what we want? emaste: is there not a WARNS= we can use somehow that gets (most of) what we want?
		smhAuthorUnsubmitted Done Inline Actions I didn't think so but on closer inspection WARNS = 6 is close, I'll give that a shot, see what the results of that are. smh: I didn't think so but on closer inspection WARNS = 6 is close, I'll give that a shot, see what…
		smhAuthorUnsubmitted Not Done Inline Actions This turned out not to be easily possible due to fixes require to dependences for WARN >= 2, so for the time being keeping it local. smh: This turned out not to be easily possible due to fixes require to dependences for WARN >= 2, so…

		# EFI_GUID assignments
		CWARNFLAGS+= -Wno-missing-braces
		# zfsimpl.c
		CWARNFLAGS+= -Wno-unused-function -Wno-array-bounds

	.include <bsd.own.mk>	.include <bsd.own.mk>
		.include <src.opts.mk>

	# In-tree GCC does not support __attribute__((ms_abi)).	# In-tree GCC does not support __attribute__((ms_abi)).
	.if ${COMPILER_TYPE} != "gcc"	.if ${COMPILER_TYPE} != "gcc"
Context not available.
	INTERNALPROG=	INTERNALPROG=

	# architecture-specific loader code	# architecture-specific loader code
	SRCS= boot1.c self_reloc.c start.S	SRCS= boot1.c self_reloc.c start.S ufs_module.c
		.if ${MK_ZFS} != "no"
		emasteUnsubmitted Done Inline Actions Aside, I wonder if we can do MK_UFS eventually? emaste: Aside, I wonder if we can do MK_UFS eventually?
		SRCS+= zfs_module.c
		.endif

		smhAuthorUnsubmitted Not Done Inline Actions Possibly, but quite a bit relies on it so its going to be none trivial. smh: Possibly, but quite a bit relies on it so its going to be none trivial.
	CFLAGS+= -I.	CFLAGS+= -I.
	CFLAGS+= -I${.CURDIR}/../include	CFLAGS+= -I${.CURDIR}/../include
Context not available.
	CFLAGS+= -I${.CURDIR}/../include/${MACHINE}	CFLAGS+= -I${.CURDIR}/../include/${MACHINE}
	CFLAGS+= -I${.CURDIR}/../../../contrib/dev/acpica/include	CFLAGS+= -I${.CURDIR}/../../../contrib/dev/acpica/include
	CFLAGS+= -I${.CURDIR}/../../..	CFLAGS+= -I${.CURDIR}/../../..
		.if ${MK_ZFS} != "no"
		CFLAGS+= -I${.CURDIR}/../../zfs/
		CFLAGS+= -I${.CURDIR}/../../../cddl/boot/zfs/
		CFLAGS+= -DZFS_EFI_BOOT
		.endif

	# Always add MI sources and REGULAR efi loader bits	# Always add MI sources and REGULAR efi loader bits
	.PATH: ${.CURDIR}/../loader/arch/${MACHINE}	.PATH: ${.CURDIR}/../loader/arch/${MACHINE}
Context not available.
		emasteUnsubmitted Done Inline Actions I wouldn't list these all here since they'll get out of date if any changes are made. just the "string and memory functions" text explains the use. emaste: I wouldn't list these all here since they'll get out of date if any changes are made. just the…