summaryrefslogtreecommitdiff
path: root/mk/check/check-files.mk
diff options
context:
space:
mode:
authorjlam <jlam@pkgsrc.org>2006-06-03 23:11:42 +0000
committerjlam <jlam@pkgsrc.org>2006-06-03 23:11:42 +0000
commite5eb2c56afcaf66f9ed3c7848573a343132b82b0 (patch)
treebfe84ef2dbbdd59b31ae6e0df262c47f9ad1b342 /mk/check/check-files.mk
parent318cea5cbda9aa49ead151bb7ecfa2b8c8d51632 (diff)
downloadpkgsrc-e5eb2c56afcaf66f9ed3c7848573a343132b82b0.tar.gz
First pass at implementing support for package system flavors other
than pkgsrc's current one. This is an important lead-up to any project that redesigns the pkg_* tools in that it doesn't tie us to past design (mis)choices. This commit mostly deals with rearranging code, although there was a considerable amount of rewriting done in cases where I thought the code was somewhat messy and was difficult to understand. The design I chose for supporting multiple package system flavors is that the various depends, install, package, etc. modules would define default targets and variables that may be overridden in files from pkgsrc/mk/flavor/${PKG_FLAVOR}. The default targets would do the sensible thing of doing nothing, and pkgsrc infrastructure would rely on the appropriate things to be defined in pkgsrc/mk/flavor to do the real work. The pkgsrc/mk/flavor directory contains subdirectories corresponding to each package system flavor that we support. Currently, I only have "pkg" which represents the current pkgsrc-native package flavor. I've separated out most of the code where we make assumptions about the package system flavor, mostly either because we directly use the pkg_* tools, or we make assumptions about the package meta-data directory, or we directly manipulate the package meta-data files, and placed it into pkgsrc/mk/flavor/pkg. There are several new modules that have been refactored out of bsd.pkg.mk as part of these changes: check, depends, install, package, and update. Each of these modules has been slimmed down by rewriting them to avoid some recursive make calls. I've also religiously documented which targets are "public" and which are "private" so that users won't rely on reaching into pkgsrc innards to call a private target. The "depends" module is a complete overhaul of the way that we handle dependencies. There is now a separate "depends" phase that occurs before the "extract" phase where dependencies are installed. This differs from the old way where dependencies were installed just before extraction occurred. The reduce-depends.mk file is now replaced by a script that is invoked only once during the depends phase and is used to generate a cookie file that holds the full set of reduced dependencies. It is now possible to type "make depends" in a package directory and all missing dependencies will be installed. Future work on this project include: * Resolve the workflow design in anticipation of future work on staged installations where "package" conceptually happens before "install". * Rewrite the buildlink3 framework to not assume the use of the pkgsrc pkg_* tools. * Rewrite the pkginstall framework to provide a standard pkg_* tool to perform the actions, and allowing a purely declarative file per package to describe what actions need to be taken at install or deinstall time. * Implement support for the SVR4 package flavor. This will be proof that the appropriate abstractions are in place to allow using a completely different set of package management tools.
Diffstat (limited to 'mk/check/check-files.mk')
-rw-r--r--mk/check/check-files.mk313
1 files changed, 313 insertions, 0 deletions
diff --git a/mk/check/check-files.mk b/mk/check/check-files.mk
new file mode 100644
index 00000000000..7d11838c4b3
--- /dev/null
+++ b/mk/check/check-files.mk
@@ -0,0 +1,313 @@
+# $NetBSD: check-files.mk,v 1.1 2006/06/03 23:11:42 jlam Exp $
+
+.if defined(PKG_DEVELOPER)
+CHECK_FILES?= yes
+.endif
+
+CHECK_FILES?= no
+CHECK_FILES_STRICT?= no
+
+###########################################################################
+# CHECK_FILES_SKIP is a list of file names that will be skipped when
+# analyzing file lists in the check-files target. This is useful to
+# avoid getting errors triggered by changes in directories not really
+# handled by pkgsrc.
+
+# Info index files updated when a new info file is added.
+.if defined(INFO_FILES)
+CHECK_FILES_SKIP+= ${PREFIX}/.*/dir$$$$
+.endif
+
+# Perl's perllocal.pod index that is regenerated when a local module
+# is added.
+#
+CHECK_FILES_SKIP+= ${PERL5_INSTALLARCHLIB}/perllocal.pod
+
+# R's index files that are regenerated when a local module
+# is added.
+#
+CHECK_FILES_SKIP+= ${PREFIX}/lib/R/doc/html/packages.html
+CHECK_FILES_SKIP+= ${PREFIX}/lib/R/doc/html/search/index.txt
+
+# We don't care about what's under /proc in Linux emulation, which is
+# just holds run-time generated data.
+#
+CHECK_FILES_SKIP+= ${PREFIX}/emul/linux/proc.*
+
+# The reference-count meta-data directory used by the pkginstall framework.
+CHECK_FILES_SKIP+= ${PKG_DBDIR}.refcount.*
+
+# File that are outside of ${PREFIX} in directories we already know we'll
+# be using for mutable data.
+#
+.for d in ${MAKE_DIRS} ${OWN_DIRS}
+CHECK_FILES_SKIP+= ${d:C/^([^\/])/${PREFIX}\/\1/}.*
+.endfor
+.for d o g m in ${MAKE_DIRS_PERMS} ${OWN_DIRS_PERMS}
+CHECK_FILES_SKIP+= ${d:C/^([^\/])/${PREFIX}\/\1/}.*
+.endfor
+
+# Mutable X11 font database files
+.if (defined(FONTS_DIRS.ttf) && !empty(FONTS_DIRS.ttf:M*)) || \
+ (defined(FONTS_DIRS.type1) && !empty(FONTS_DIRS.type1:M*)) || \
+ (defined(FONTS_DIRS.x11) && !empty(FONTS_DIRS.x11:M*))
+CHECK_FILES_SKIP+= ${PREFIX}/.*/fonts.alias
+CHECK_FILES_SKIP+= ${PREFIX}/.*/fonts.dir
+CHECK_FILES_SKIP+= ${PREFIX}/.*/fonts.scale
+.endif
+
+# Mutable charset.alias file
+CHECK_FILES_SKIP+= ${PREFIX}/lib/charset.alias
+
+_CHECK_FILES_SKIP_FILTER= ${GREP} -vx ${CHECK_FILES_SKIP:@f@-e ${f:Q}@}
+
+###########################################################################
+# These are the files generated and used by the check-files implementation
+# used across several check-files targets.
+#
+# _CHECK_FILES_COOKIE.* are the cookie files that contain the error
+# messages discovered during each stage of file-checking.
+#
+# _CHECK_FILES_PRE.* are the file lists generated before any files
+# from the package are installed.
+#
+# _CHECK_FILES_POST.* are the file lists generated after all files
+# from the package are installed.
+#
+# The "pre" and "post" file lists are compared against each other to
+# determine if the package is installing files where it shouldn't be.
+#
+_CHECK_FILES_COOKIE.prefix= ${WRKDIR}/.check_files_prefix_cookie
+_CHECK_FILES_PRE.prefix= ${WRKDIR}/.prefix.pre
+_CHECK_FILES_POST.prefix= ${WRKDIR}/.prefix.post
+
+_CHECK_FILES_COOKIE.sysconfdir= ${WRKDIR}/.check_files_sysconfdir_cookie
+_CHECK_FILES_PRE.sysconfdir= ${WRKDIR}/.sysconfdir.pre
+_CHECK_FILES_POST.sysconfdir= ${WRKDIR}/.sysconfdir.post
+
+_CHECK_FILES_COOKIE.varbase= ${WRKDIR}/.check_files_varbase_cookie
+_CHECK_FILES_PRE.varbase= ${WRKDIR}/.varbase.pre
+_CHECK_FILES_POST.varbase= ${WRKDIR}/.varbase.post
+
+_CHECK_FILES_COOKIES= # empty
+_CHECK_FILES_COOKIES+= ${_CHECK_FILES_COOKIE.prefix}
+.if empty(CHECK_FILES_STRICT:M[nN][oO])
+_CHECK_FILES_COOKIES+= ${_CHECK_FILES_COOKIE.sysconfdir}
+_CHECK_FILES_COOKIES+= ${_CHECK_FILES_COOKIE.varbase}
+.endif
+
+###########################################################################
+# _CHECK_FILES_PRE holds the list of targets that are built as part of
+# building the check-files-pre target. These targets should cause the
+# "pre" file lists to be generated.
+#
+_CHECK_FILES_PRE= check-files-pre-message
+_CHECK_FILES_PRE+= ${_CHECK_FILES_PRE.prefix}
+.if empty(CHECK_FILES_STRICT:M[nN][oO])
+_CHECK_FILES_PRE+= ${_CHECK_FILES_PRE.sysconfdir}
+_CHECK_FILES_PRE+= ${_CHECK_FILES_PRE.varbase}
+.endif
+
+###########################################################################
+# _CHECK_FILES_POST holds the list of targets that are built as part of
+# building the check-files-post target. These targets should cause the
+# "post" file lists to be generated.
+#
+_CHECK_FILES_POST= check-files-post-message
+_CHECK_FILES_POST+= ${_CHECK_FILES_POST.prefix}
+.if empty(CHECK_FILES_STRICT:M[nN][oO])
+_CHECK_FILES_POST+= ${_CHECK_FILES_POST.sysconfdir}
+_CHECK_FILES_POST+= ${_CHECK_FILES_POST.varbase}
+.endif
+
+###########################################################################
+# check-files-pre and check-files-post targets and subtargets
+#
+.PHONY: check-files-pre check-files-post
+check-files-pre: ${_CHECK_FILES_PRE}
+check-files-post: ${_CHECK_FILES_POST}
+
+.PHONY: check-files-pre-message check-files-post-message
+check-files-pre-message:
+ ${_PKG_SILENT}${_PKG_DEBUG}${STEP_MSG} \
+ "Generating pre-install file lists"
+
+check-files-post-message:
+ ${_PKG_SILENT}${_PKG_DEBUG}${STEP_MSG} \
+ "Generating post-install file lists"
+
+${_CHECK_FILES_PRE.prefix} ${_CHECK_FILES_POST.prefix}:
+ ${_PKG_SILENT}${_PKG_DEBUG} \
+ ${FIND} ${PREFIX}/. \( -type f -o -type l \) -print 2>/dev/null \
+ | ${SED} -e 's,/\./,/,' \
+ | ${_CHECK_FILES_SKIP_FILTER} > ${.TARGET} \
+ || ${TRUE}
+
+${_CHECK_FILES_PRE.sysconfdir} ${_CHECK_FILES_POST.sysconfdir}:
+ ${_PKG_SILENT}${_PKG_DEBUG} \
+ ${FIND} ${PKG_SYSCONFDIR}/. -print 2>/dev/null \
+ | ${SED} -e 's,/\./,/,' \
+ | ${_CHECK_FILES_SKIP_FILTER} > ${.TARGET} \
+ || ${TRUE}
+
+${_CHECK_FILES_PRE.varbase} ${_CHECK_FILES_POST.varbase}:
+ ${_PKG_SILENT}${_PKG_DEBUG} \
+ ${FIND} ${VARBASE}/. -print 2>/dev/null \
+ | ${SED} -e 's,/\./,/,' \
+ | ${_CHECK_FILES_SKIP_FILTER} > ${.TARGET} \
+ || ${TRUE}
+
+###########################################################################
+# check-files targets and subtargets
+#
+.PHONY: check-files-prefix check-files-sysconfdir check-files-varbase
+check-files-prefix: ${_CHECK_FILES_COOKIE.prefix}
+check-files-sysconfdir: ${_CHECK_FILES_COOKIE.sysconfdir}
+check-files-varbase: ${_CHECK_FILES_COOKIE.varbase}
+
+# The check-files target looks at the cookie files generated by the
+# check-files-* subtargets, and if they are non-empty, then they
+# contain the error messages collected during the build of each
+# subtarget.
+#
+.PHONY: check-files
+check-files: ${_CHECK_FILES_COOKIES}
+ ${_PKG_SILENT}${_PKG_DEBUG}${STEP_MSG} \
+ "Checking file-check results for ${PKGNAME}"
+.for _cookie_ in ${_CHECK_FILES_COOKIES}
+ ${_PKG_SILENT}${_PKG_DEBUG}${CAT} ${_cookie_}
+.endfor
+ ${_PKG_SILENT}${_PKG_DEBUG} \
+ ${CAT} ${_CHECK_FILES_COOKIES} | ${AWK} 'END { if (NR > 0) exit 1; }'
+
+# Check ${PREFIX} for files which are not listed in the generated ${PLIST}
+# and vice-versa.
+#
+${_CHECK_FILES_COOKIE.prefix}:
+.if !defined(NO_PKG_REGISTER)
+ ${_PKG_SILENT}${_PKG_DEBUG} \
+ if ${TEST} ! -f ${_CHECK_FILES_PRE.prefix} -o \
+ ! -f ${_CHECK_FILES_POST.prefix}; \
+ then \
+ { exit 0; }; \
+ fi; \
+ f_added=${WRKDIR}/.files.added; \
+ f_deleted=${WRKDIR}/.files.deleted; \
+ f_expected=${WRKDIR}/.files.expected; \
+ f_missing=${WRKDIR}/.files.missing; \
+ f_missing_real=${WRKDIR}/.files.missing.real; \
+ f_missing_skip=${WRKDIR}/.files.missing.skip; \
+ f_extra=${WRKDIR}/.files.extra; \
+ ${DIFF} -u ${_CHECK_FILES_PRE.prefix} ${_CHECK_FILES_POST.prefix} \
+ > ${WRKDIR}/.files.diff || ${TRUE}; \
+ ${GREP} '^+/' ${WRKDIR}/.files.diff | ${SED} "s|^+||" | ${SORT} \
+ > $$f_added; \
+ ${GREP} '^-/' ${WRKDIR}/.files.diff | ${SED} "s|^-||" | ${SORT} \
+ > $$f_deleted; \
+ ${GREP} '^[A-Za-z]' ${PLIST} | ${SED} "s|^|${PREFIX}/|" | ${SORT} \
+ > $$f_expected; \
+ ${DIFF} -u ${WRKDIR}/.files.expected ${WRKDIR}/.files.added \
+ | ${GREP} '^-[^-]' | ${SED} "s|^-||" \
+ > $$f_missing; \
+ ${DIFF} -u ${WRKDIR}/.files.expected ${WRKDIR}/.files.added \
+ | ${GREP} '^+[^+]' | ${SED} "s|^+||" \
+ > $$f_extra; \
+ if ${AWK} 'END { if (NR == 0) exit 1; }' $$f_deleted; then \
+ { ${ECHO} "*** The following files have been deleted" \
+ "from ${PREFIX}!"; \
+ ${SED} "s|^| |" $$f_deleted; \
+ } > ${.TARGET}; \
+ fi; \
+ if ${AWK} 'END { if (NR == 0) exit 1; }' $$f_missing $$f_extra; \
+ then \
+ { ${ECHO} "*** The PLIST does not match installed files!"; \
+ ${CAT} $$f_missing | ${_CHECK_FILES_SKIP_FILTER} \
+ > $$f_missing_real; \
+ ${DIFF} -u $$f_missing $$f_missing_real \
+ | ${GREP} '^-[^-]' | ${SED} "s|^-||" \
+ > $$f_missing_skip; \
+ if ${AWK} 'END { if (NR == 0) exit 1; }' $$f_missing_real; \
+ then \
+ ${ECHO} "*** The following files are in the" \
+ "PLIST but not in ${PREFIX}:"; \
+ ${SED} "s|^| |" $$f_missing_real; \
+ fi; \
+ if ${AWK} 'END { if (NR == 0) exit 1; }' $$f_extra; then \
+ ${ECHO} "*** The following files are in" \
+ "${PREFIX} but not in the PLIST:"; \
+ ${SED} "s|^| |" $$f_extra; \
+ fi; \
+ if ${AWK} 'END { if (NR == 0) exit 1; }' $$f_missing_skip; \
+ then \
+ ${ECHO} "*** The following files are in both" \
+ "the PLIST and CHECK_FILES_SKIP:"; \
+ ${SED} "s|^| |" $$f_missing_skip; \
+ fi; \
+ } >> ${.TARGET}; \
+ fi; \
+ ${RM} -f ${WRKDIR}/.files.diff $$f_added $$f_deleted \
+ $$f_expected $$f_missing $$f_extra \
+ $$f_missing_real $$f_missing_skip
+.endif
+ ${_PKG_SILENT}${_PKG_DEBUG}${TOUCH} ${TOUCH_FLAGS} ${.TARGET}
+
+# Check ${SYSCONFDIR} for files which are not in the PLIST and are also
+# not copied into place by the INSTALL scripts.
+#
+${_CHECK_FILES_COOKIE.sysconfdir}:
+.if !defined(NO_PKG_REGISTER)
+ ${_PKG_SILENT}${_PKG_DEBUG} \
+ if ${TEST} ! -f ${_CHECK_FILES_PRE.sysconfdir} -o \
+ ! -f ${_CHECK_FILES_POST.sysconfdir}; \
+ then \
+ { exit 0; }; \
+ fi; \
+ if ! ${CMP} -s ${_CHECK_FILES_PRE.sysconfdir} \
+ ${_CHECK_FILES_POST.sysconfdir}; \
+ then \
+ { ${ECHO} "*** The package has modified ${PKG_SYSCONFDIR}" \
+ "contents directly!"; \
+ ${ECHO} " The offending files/directories are:"; \
+ ${DIFF} -u ${_CHECK_FILES_PRE.sysconfdir} \
+ ${_CHECK_FILES_POST.sysconfdir} \
+ | ${GREP} '^+[^+]' | ${SED} "s|^+| |"; \
+ } > ${.TARGET}; \
+ fi
+.endif
+ ${_PKG_SILENT}${_PKG_DEBUG}${TOUCH} ${TOUCH_FLAGS} ${.TARGET}
+
+# Check ${VARBASE} for files which are not in the PLIST and are also
+# not created by the INSTALL scripts.
+#
+${_CHECK_FILES_COOKIE.varbase}:
+.if !defined(NO_PKG_REGISTER)
+ ${_PKG_SILENT}${_PKG_DEBUG} \
+ if ${TEST} ! -f ${_CHECK_FILES_PRE.varbase} -o \
+ ! -f ${_CHECK_FILES_POST.varbase}; \
+ then \
+ { exit 0; }; \
+ fi; \
+ if ! ${CMP} -s ${_CHECK_FILES_PRE.varbase} \
+ ${_CHECK_FILES_POST.varbase}; \
+ then \
+ { ${ECHO} "*** The package has modified ${VARBASE}" \
+ "contents directly!"; \
+ ${ECHO} " The offending files/directories are:"; \
+ ${DIFF} -u ${_CHECK_FILES_PRE.varbase} \
+ ${_CHECK_FILES_POST.varbase} \
+ | ${GREP} '^+[^+]' | ${SED} "s|^+| |"; \
+ } > ${.TARGET}; \
+ fi
+.endif
+ ${_PKG_SILENT}${_PKG_DEBUG}${TOUCH} ${TOUCH_FLAGS} ${.TARGET}
+
+###########################################################################
+# check-files-clean removes the state files related to the "check-files"
+# target so that the check-files-{pre,post} targets may be re-run.
+#
+.PHONY: check-files-clean
+check-clean: check-files-clean
+check-files-clean:
+ ${_PKG_SILENT}${_PKG_DEBUG} \
+ ${RM} -f ${_CHECK_FILES_COOKIES} \
+ ${_CHECK_FILES_PRE} ${_CHECK_FILES_POST}