git: b58ea3e1f777 - main - Fix hand-rolled METALOG entries for installconfig during distributeworld
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Mon, 28 Feb 2022 22:41:06 UTC
The branch main has been updated by jrtc27: URL: https://cgit.FreeBSD.org/src/commit/?id=b58ea3e1f7779aac4eebcec5dffc9ebef203e583 commit b58ea3e1f7779aac4eebcec5dffc9ebef203e583 Author: Jessica Clarke <jrtc27@FreeBSD.org> AuthorDate: 2022-02-28 22:36:39 +0000 Commit: Jessica Clarke <jrtc27@FreeBSD.org> CommitDate: 2022-02-28 22:36:39 +0000 Fix hand-rolled METALOG entries for installconfig during distributeworld During distributeworld we call distribute on subdirectories, which in turn calls installconfig. However, this recursive installconfig call appends the distribution name (in these cases, "base") to DESTDIR. For install(1) this works fine as its -D argument comes from the top-level Makefile.inc1, which passes the original DESTDIR, thereby resulting in the METALOG entry having the distribution name as a prefix representing its true installed path relative to the root, but for the hand-rolled entries they do not use install(1) and thus do not have access to what the original DESTDIR was, resulting in the METALOG missing this prefix. Thus, pass down the name of the distribution via a new variable DISTBASE (chosen as Makefile.inc1 already uses that to convey this exact same information to etc's distrib-dirs during distributeworld) and prepend this to the handful of manually-generated METALOG entries. For the installworld case this variable will be empty and so this behaves as before. Note that we need to be careful to avoid double slashes in the METALOG; distributeworld uses find | awk to split the single METALOG up into multiple dist.meta files, and this relies on the paths in the METALOG having the exact prefix ./dist (or ./dist/usr/lib/debug). Reviewed by: brooks, emaste Differential Revision: https://reviews.freebsd.org/D33997 --- lib/libc/gen/Makefile.inc | 6 +++--- share/mk/bsd.subdir.mk | 2 +- usr.bin/login/Makefile | 2 +- usr.sbin/services_mkdb/Makefile | 2 +- 4 files changed, 6 insertions(+), 6 deletions(-) diff --git a/lib/libc/gen/Makefile.inc b/lib/libc/gen/Makefile.inc index fd8a0feeb748..395edefc0503 100644 --- a/lib/libc/gen/Makefile.inc +++ b/lib/libc/gen/Makefile.inc @@ -573,8 +573,8 @@ install-passwd: .PHONY ${PWD_MKDB_CMD} -i -p -d ${DESTDIR}/etc ${DESTDIR}/etc/master.passwd .if defined(NO_ROOT) && defined(METALOG) ( \ - echo "./etc/pwd.db type=file mode=0644 uname=root gname=wheel"; \ - echo "./etc/spwd.db type=file mode=0600 uname=root gname=wheel"; \ - echo "./etc/passwd type=file mode=0644 uname=root gname=wheel"; \ + echo ".${DISTBASE}/etc/pwd.db type=file mode=0644 uname=root gname=wheel"; \ + echo ".${DISTBASE}/etc/spwd.db type=file mode=0600 uname=root gname=wheel"; \ + echo ".${DISTBASE}/etc/passwd type=file mode=0644 uname=root gname=wheel"; \ ) | cat -l >> ${METALOG} .endif diff --git a/share/mk/bsd.subdir.mk b/share/mk/bsd.subdir.mk index 91c114f8d361..fb686a101322 100644 --- a/share/mk/bsd.subdir.mk +++ b/share/mk/bsd.subdir.mk @@ -94,7 +94,7 @@ DISTRIBUTION?= base distribute: .MAKE .for dist in ${DISTRIBUTION} ${_+_}cd ${.CURDIR}; \ - ${MAKE} install installconfig -DNO_SUBDIR DESTDIR=${DISTDIR}/${dist} SHARED=copies + ${MAKE} install installconfig -DNO_SUBDIR DISTBASE=/${dist} DESTDIR=${DISTDIR}/${dist} SHARED=copies .endfor .endif # Convenience targets to run 'build${target}' and 'install${target}' when diff --git a/usr.bin/login/Makefile b/usr.bin/login/Makefile index c876434dcc2e..f8e62d4f934f 100644 --- a/usr.bin/login/Makefile +++ b/usr.bin/login/Makefile @@ -30,7 +30,7 @@ PRECIOUSPROG= afterinstallconfig: ${CAP_MKDB_CMD} ${CAP_MKDB_ENDIAN} ${DESTDIR}/etc/login.conf .if defined(NO_ROOT) && defined(METALOG) - echo "./etc/login.conf.db type=file mode=0644 uname=root gname=wheel" | \ + echo ".${DISTBASE}/etc/login.conf.db type=file mode=0644 uname=root gname=wheel" | \ cat -l >> ${METALOG} .endif diff --git a/usr.sbin/services_mkdb/Makefile b/usr.sbin/services_mkdb/Makefile index 9d728b6da824..c1edc37452ab 100644 --- a/usr.sbin/services_mkdb/Makefile +++ b/usr.sbin/services_mkdb/Makefile @@ -17,7 +17,7 @@ afterinstallconfig: ${SERVICES_MKDB_CMD} ${CAP_MKDB_ENDIAN} -q -o ${DESTDIR}/var/db/services.db \ ${DESTDIR}/etc/services .if defined(NO_ROOT) && defined(METALOG) - echo "./var/db/services.db type=file mode=0644 uname=root gname=wheel" | \ + echo ".${DISTBASE}/var/db/services.db type=file mode=0644 uname=root gname=wheel" | \ cat -l >> ${METALOG} .endif .endif