Changeset View
Changeset View
Standalone View
Standalone View
sys/boot/efi/boot1/Makefile
# $FreeBSD$ | # $FreeBSD$ | ||||
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 \ | |||||
emaste: Can you explain why individual warning flags are turned on here? | |||||
smhAuthorUnsubmitted Done Inline ActionsAll the flags from kern.mk smh: All the flags from kern.mk | |||||
Done Inline Actionsis 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? | |||||
Done Inline ActionsI 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… | |||||
Not Done Inline ActionsThis 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" | ||||
MK_SSP= no | MK_SSP= no | ||||
PROG= boot1.sym | PROG= boot1.sym | ||||
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" | |||||
SRCS+= zfs_module.c | |||||
.endif | |||||
Done Inline ActionsAside, I wonder if we can do MK_UFS eventually? emaste: Aside, I wonder if we can do MK_UFS eventually? | |||||
Not Done Inline ActionsPossibly, 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 | ||||
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}/../../.. | ||||
CFLAGS+= -DEFI_UFS_BOOT | |||||
.if ${MK_ZFS} != "no" | |||||
CFLAGS+= -I${.CURDIR}/../../zfs/ | |||||
CFLAGS+= -I${.CURDIR}/../../../cddl/boot/zfs/ | |||||
CFLAGS+= -DEFI_ZFS_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} | ||||
.PATH: ${.CURDIR}/../loader | .PATH: ${.CURDIR}/../loader | ||||
.PATH: ${.CURDIR}/../../common | .PATH: ${.CURDIR}/../../common | ||||
CFLAGS+= -I${.CURDIR}/../../common | CFLAGS+= -I${.CURDIR}/../../common | ||||
FILES= boot1.efi boot1.efifat | FILES= boot1.efi boot1.efifat | ||||
Show All 10 Lines | |||||
LDFLAGS+= -Wl,-znocombreloc | LDFLAGS+= -Wl,-znocombreloc | ||||
.endif | .endif | ||||
.if ${MACHINE_CPUARCH} == "arm" || ${MACHINE_CPUARCH} == "i386" | .if ${MACHINE_CPUARCH} == "arm" || ${MACHINE_CPUARCH} == "i386" | ||||
# | # | ||||
# Add libstand for the runtime functions used by the compiler - for example | # Add libstand for the runtime functions used by the compiler - for example | ||||
# __aeabi_* (arm) or __divdi3 (i386). | # __aeabi_* (arm) or __divdi3 (i386). | ||||
# | # | ||||
DPADD+= ${LIBSTAND} | DPADD+= ${LIBSTAND} | ||||
LDADD+= -lstand | LDADD+= -lstand | ||||
Done Inline ActionsI 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… | |||||
.endif | .endif | ||||
DPADD+= ${LDSCRIPT} | DPADD+= ${LDSCRIPT} | ||||
OBJCOPY?= objcopy | OBJCOPY?= objcopy | ||||
OBJDUMP?= objdump | OBJDUMP?= objdump | ||||
.if ${MACHINE_CPUARCH} == "amd64" | .if ${MACHINE_CPUARCH} == "amd64" | ||||
▲ Show 20 Lines • Show All 51 Lines • Show Last 20 Lines |
Can you explain why individual warning flags are turned on here?