diff options
author | jlam <jlam@pkgsrc.org> | 2005-05-09 05:06:55 +0000 |
---|---|---|
committer | jlam <jlam@pkgsrc.org> | 2005-05-09 05:06:55 +0000 |
commit | 03e9337879dfeaee903029eb135746d04398bfdc (patch) | |
tree | 3bff071687c8b973f251436d751c2dc954d88ba6 /databases/openldap | |
parent | c4933146fb49fa4fa4b5c4e1569df0faa19285be (diff) | |
download | pkgsrc-03e9337879dfeaee903029eb135746d04398bfdc.tar.gz |
Teach bsd.pkg.mk to create a phase-specific "makevars.mk" file that
caches variable definitions that were computed by make. These variables
are specified by listing them in MAKE_VARS, e.g.,
.if !defined(FOO)
FOO!= very_time_consuming_command
.endif
MAKE_VARS+= FOO
bsd.pkg.mk will include only the one generated during the most recent
phase. A particular phase's makevars.mk file consists of variable
definitions that are a superset of all of the ones produced in previous
phases of the build.
The caching is useful because bsd.pkg.mk invokes make recursively,
which in the example above has the potential to run the very time-consuming
command each time unless we cause FOO to be defined for the sub-make
processes. We don't cache via MAKE_FLAGS because MAKE_FLAGS isn't
consistently applied to every invocation of make, and also because
MAKE_FLAGS can overflow the maximum length of a make variable very
quickly if we add many values to it.
One important and desirable property of variables cached via MAKE_VARS
is that they only apply to the current package, and not to any
dependencies whose builds may have been triggered by the current
package.
The makevars.mk files are generated by new targets fetch-vars,
extract-vars, patch-vars, etc., and these targets are built during
the corresponding real-* target to ensure that they are being invoked
with PKG_PHASE set to the proper value.
Also, remove the variables cache file that bsd.wrapper.mk was generating
since the new makevars.mk files provide the same functionality at a
higher level. Change all WRAPPER_VARS definitions that were used by
the old wrapper-phase cache file into MAKE_VARS definitions.
Diffstat (limited to 'databases/openldap')
-rw-r--r-- | databases/openldap/buildlink3.mk | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/databases/openldap/buildlink3.mk b/databases/openldap/buildlink3.mk index ee6a588373b..b6fd1e810b3 100644 --- a/databases/openldap/buildlink3.mk +++ b/databases/openldap/buildlink3.mk @@ -1,4 +1,4 @@ -# $NetBSD: buildlink3.mk,v 1.16 2005/05/08 12:03:56 jlam Exp $ +# $NetBSD: buildlink3.mk,v 1.17 2005/05/09 05:06:55 jlam Exp $ BUILDLINK_DEPTH:= ${BUILDLINK_DEPTH}+ OPENLDAP_BUILDLINK3_MK:= ${OPENLDAP_BUILDLINK3_MK}+ @@ -22,9 +22,9 @@ BUILDLINK_PKGSRCDIR.openldap?= ../../databases/openldap PKG_OPTIONS.openldap!= \ cd ${BUILDLINK_PKGSRCDIR.openldap} && \ ${MAKE} show-var ${MAKE_FLAGS} VARNAME=PKG_OPTIONS -MAKE_FLAGS+= PKG_OPTIONS.openldap=${PKG_OPTIONS.openldap:Q} -WRAPPER_VARS+= PKG_OPTIONS.openldap +MAKE_FLAGS+= PKG_OPTIONS.openldap=${PKG_OPTIONS.openldap:Q} .endif +MAKE_VARS+= PKG_OPTIONS.openldap .if !empty(PKG_OPTIONS.openldap:Mkerberos) || \ !empty(PKG_OPTIONS.openldap:Msasl) |