Module Name:	pkgsrc
Committed By:	jlam
Date:		Mon May  9 05:06:56 UTC 2005

Modified Files:
	pkgsrc/databases/openldap: buildlink3.mk
	pkgsrc/devel/apr: buildlink3.mk
	pkgsrc/misc/kdepim3: buildlink3.mk
	pkgsrc/mk: bsd.pkg.mk bsd.prefs.mk
	pkgsrc/mk/buildlink3: bsd.buildlink3.mk
	pkgsrc/mk/wrapper: bsd.wrapper.mk
	pkgsrc/security/heimdal: buildlink3.mk
	pkgsrc/security/openssl: buildlink3.mk
	pkgsrc/x11/kdebase3: buildlink3.mk
	pkgsrc/x11/kdelibs3: buildlink3.mk

Log Message:
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.


To generate a diff of this commit:
cvs rdiff -r1.16 -r1.17 pkgsrc/databases/openldap/buildlink3.mk
cvs rdiff -r1.14 -r1.15 pkgsrc/devel/apr/buildlink3.mk
cvs rdiff -r1.7 -r1.8 pkgsrc/misc/kdepim3/buildlink3.mk
cvs rdiff -r1.1630 -r1.1631 pkgsrc/mk/bsd.pkg.mk
cvs rdiff -r1.188 -r1.189 pkgsrc/mk/bsd.prefs.mk
cvs rdiff -r1.168 -r1.169 pkgsrc/mk/buildlink3/bsd.buildlink3.mk
cvs rdiff -r1.28 -r1.29 pkgsrc/mk/wrapper/bsd.wrapper.mk
cvs rdiff -r1.21 -r1.22 pkgsrc/security/heimdal/buildlink3.mk
cvs rdiff -r1.24 -r1.25 pkgsrc/security/openssl/buildlink3.mk
cvs rdiff -r1.7 -r1.8 pkgsrc/x11/kdebase3/buildlink3.mk
cvs rdiff -r1.8 -r1.9 pkgsrc/x11/kdelibs3/buildlink3.mk

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.