Changeset View
Changeset View
Standalone View
Standalone View
lib/libsysdecode/Makefile
Show First 20 Lines • Show All 118 Lines • ▼ Show 20 Lines | |||||
# Workaround duplicate declarations in <netinet/ip_compat.h> | # Workaround duplicate declarations in <netinet/ip_compat.h> | ||||
CFLAGS.gcc.ioctl.c+= -Wno-redundant-decls | CFLAGS.gcc.ioctl.c+= -Wno-redundant-decls | ||||
CFLAGS.gcc+= ${CFLAGS.gcc.${.IMPSRC}} | CFLAGS.gcc+= ${CFLAGS.gcc.${.IMPSRC}} | ||||
DEPENDOBJS+= tables.h | DEPENDOBJS+= tables.h | ||||
incdir=${SYSROOT:U${DESTDIR}}${INCLUDEDIR} | incdir=${SYSROOT:U${DESTDIR}}${INCLUDEDIR} | ||||
tables.h: mktables ${incdir}/netinet/in.h ${incdir}/netinet/tcp.h \ | .if !defined(_SKIP_BUILD) | ||||
${incdir}/netinet6/in6.h | tblhdrs!=sh ${.CURDIR}/mktables -d ${incdir} | sort -u | ||||
.endif | |||||
tables.h: mktables ${tblhdrs:S?^?${incdir}/?} | |||||
emaste: @bdrewery does this need to be wrapped in a `.if !defined(_SKIP_BUILD)`? | |||||
Not Done Inline ActionsI think it does, but I'm far from certain. imp: I think it does, but I'm far from certain. | |||||
Done Inline ActionsIt should only be a perf issue if not there I think, but some testing (to the extent that I understand this part of the build system) suggests it works with a _SKIP_BUILD test here. emaste: It should only be a perf issue if not there I think, but some testing (to the extent that I… | |||||
Not Done Inline ActionsYes please. _SKIP_BUILD is for some treewalks that are not building anything so don't need to know dependencies. Having said that I don't see this this change is needed. There's already a .depend file here for this exact thing. rS308602 was the start of that. bdrewery: Yes please. `_SKIP_BUILD` is for some treewalks that are not building anything so don't need to… | |||||
sh ${.CURDIR}/mktables ${incdir} ${.TARGET}.tmp && \ | sh ${.CURDIR}/mktables ${incdir} ${.TARGET}.tmp && \ | ||||
mv -f ${.TARGET}.tmp ${.TARGET} | mv -f ${.TARGET}.tmp ${.TARGET} | ||||
# mkioctls runs find(1) for headers so needs to rebuild every time. This used | # mkioctls runs find(1) for headers so needs to rebuild every time. This used | ||||
# to be a hack only done in buildworld. | # to be a hack only done in buildworld. | ||||
.if !defined(_SKIP_BUILD) | .if !defined(_SKIP_BUILD) | ||||
ioctl.c.tmp: .PHONY | ioctl.c.tmp: .PHONY | ||||
.endif | .endif | ||||
Show All 12 Lines |
@bdrewery does this need to be wrapped in a .if !defined(_SKIP_BUILD)?