Changeset View
Changeset View
Standalone View
Standalone View
Mk/bsd.port.mk
Show First 20 Lines • Show All 1,138 Lines • ▼ Show 20 Lines | |||||
.elif exists(${SRC_BASE}/sys/sys/param.h) | .elif exists(${SRC_BASE}/sys/sys/param.h) | ||||
OSVERSION!= ${AWK} '/^\#define[[:blank:]]__FreeBSD_version/ {print $$3}' < ${SRC_BASE}/sys/sys/param.h | OSVERSION!= ${AWK} '/^\#define[[:blank:]]__FreeBSD_version/ {print $$3}' < ${SRC_BASE}/sys/sys/param.h | ||||
.else | .else | ||||
.error Unable to determine OS version. Either define OSVERSION, install /usr/include/sys/param.h or define SRC_BASE. | .error Unable to determine OS version. Either define OSVERSION, install /usr/include/sys/param.h or define SRC_BASE. | ||||
.endif | .endif | ||||
.endif | .endif | ||||
_EXPORTED_VARS+= OSVERSION | _EXPORTED_VARS+= OSVERSION | ||||
.if ${OPSYS} == FreeBSD && (${OSVERSION} < 1003000 || (${OSVERSION} >= 1100000 && ${OSVERSION} < 1100122)) | |||||
marino: I assume this line is what you were thinking about for DragonFly support.
DFLYVERSION is the… | |||||
_UNSUPPORTED_SYSTEM_MESSAGE= Ports Collection support for your ${OPSYS} version has ended, and no ports\ | |||||
Done Inline Actionss/to supported/to a supported/ bcr: s/to supported/to a supported/ | |||||
are guaranteed to build on this system. Please upgrade to a supported release. | |||||
. if defined(ALLOW_UNSUPPORTED_SYSTEM) | |||||
WARNING+= "${_UNSUPPORTED_SYSTEM_MESSAGE}" | |||||
Done Inline ActionsI think we can make the text simpler and clearer. I don't like suggesting that people pull an old snapshot. Then it looks like an official recommendation, and then we have to do tech support for people who have a 5 y/o ports tree who are just following instructions. I'd recommend this: Ports Collection support for your ${OPSYS} version has ended, and no ports are guaranteed to build on this system. Please upgrade to a supported release. adamw: I think we can make the text simpler and clearer. I don't like suggesting that people pull an… | |||||
. else | |||||
Not Done Inline ActionsThis should not be using IGNORE but a target in the sanity sequence ending with ${FALSE}. mat: This should not be using IGNORE but a target in the sanity sequence ending with ${FALSE}. | |||||
Not Done Inline ActionsWhy don't you put the target directly here ? mat: Why don't you put the target directly here ? | |||||
Not Done Inline ActionsHm, I'm got an idea there's specific place for these targets. AMDmi3: Hm, I'm got an idea there's specific place for these targets. | |||||
Not Done Inline ActionsYou could also move the whole block to where that place was :-) mat: You could also move the whole block to where that place was :-) | |||||
show-unsupported-system-error: | |||||
@${ECHO_MSG} "/!\\ ERROR: /!\\" | |||||
@${ECHO_MSG} | |||||
@${ECHO_MSG} "${_UNSUPPORTED_SYSTEM_MESSAGE}" | ${FMT} 75 79 | |||||
@${ECHO_MSG} | |||||
@${ECHO_MSG} "No support will be provided if you silence this message by defining ALLOW_UNSUPPORTED_SYSTEM." | ${FMT} 75 79 | |||||
@${ECHO_MSG} | |||||
@${FALSE} | |||||
Not Done Inline ActionsThat wording is awkward. I'd suggest this: No support will be provided if you silence this message by defining ALLOW_UNSUPPORTED_SYSTEM. adamw: That wording is awkward. I'd suggest this:
```No support will be provided if you silence this… | |||||
. endif | |||||
.endif | |||||
# Convert OSVERSION to major release number | # Convert OSVERSION to major release number | ||||
_OSVERSION_MAJOR= ${OSVERSION:C/([0-9]?[0-9])([0-9][0-9])[0-9]{3}/\1/} | _OSVERSION_MAJOR= ${OSVERSION:C/([0-9]?[0-9])([0-9][0-9])[0-9]{3}/\1/} | ||||
# Sanity checks for chroot/jail building. | # Sanity checks for chroot/jail building. | ||||
# Skip if OSVERSION specified on cmdline for testing. Only works for bmake. | # Skip if OSVERSION specified on cmdline for testing. Only works for bmake. | ||||
.if !defined(.MAKEOVERRIDES) || !${.MAKEOVERRIDES:MOSVERSION} | .if !defined(.MAKEOVERRIDES) || !${.MAKEOVERRIDES:MOSVERSION} | ||||
.if ${_OSVERSION_MAJOR} != ${_OSRELEASE:R} | .if ${_OSVERSION_MAJOR} != ${_OSRELEASE:R} | ||||
.error UNAME_r (${_OSRELEASE}) and OSVERSION (${OSVERSION}) do not agree on major version number. | .error UNAME_r (${_OSRELEASE}) and OSVERSION (${OSVERSION}) do not agree on major version number. | ||||
.elif ${_OSVERSION_MAJOR} != ${OSREL:R} | .elif ${_OSVERSION_MAJOR} != ${OSREL:R} | ||||
▲ Show 20 Lines • Show All 4,008 Lines • ▼ Show 20 Lines | |||||
# it depends on before running its SEQ. | # it depends on before running its SEQ. | ||||
# | # | ||||
# Main target has a priority of 500, pre-target 300, post-target 700, | # Main target has a priority of 500, pre-target 300, post-target 700, | ||||
# target-depends 150. Other targets are spaced in between those | # target-depends 150. Other targets are spaced in between those | ||||
# | # | ||||
# If you change the pre-foo and post-foo values here, go and keep them in sync | # If you change the pre-foo and post-foo values here, go and keep them in sync | ||||
# in _OPTIONS_TARGETS in bsd.options.mk | # in _OPTIONS_TARGETS in bsd.options.mk | ||||
_SANITY_SEQ= 050:post-chroot 100:pre-everything 150:check-makefile \ | _SANITY_SEQ= 050:post-chroot 100:pre-everything \ | ||||
125:show-unsupported-system-error 150:check-makefile \ | |||||
200:show-warnings 210:show-dev-warnings 220:show-dev-errors \ | 200:show-warnings 210:show-dev-warnings 220:show-dev-errors \ | ||||
250:check-categories 300:check-makevars \ | 250:check-categories 300:check-makevars \ | ||||
350:check-desktop-entries 400:check-depends \ | 350:check-desktop-entries 400:check-depends \ | ||||
450:identify-install-conflicts 500:check-deprecated \ | 450:identify-install-conflicts 500:check-deprecated \ | ||||
550:check-vulnerable 600:check-license 650:check-config \ | 550:check-vulnerable 600:check-license 650:check-config \ | ||||
700:buildanyway-message 750:options-message ${_USES_sanity} | 700:buildanyway-message 750:options-message ${_USES_sanity} | ||||
_PKG_DEP= check-sanity | _PKG_DEP= check-sanity | ||||
▲ Show 20 Lines • Show All 179 Lines • Show Last 20 Lines |
I assume this line is what you were thinking about for DragonFly support.
DFLYVERSION is the equivalent of OSVERSION, but it's only defined on Dports. Still, this should work:
Note that I put parentheses around the original condition.