diff options
| author | Matthew Seaman <matthew@FreeBSD.org> | 2024-05-27 08:14:15 +0000 |
|---|---|---|
| committer | Matthew Seaman <matthew@FreeBSD.org> | 2024-06-04 05:11:10 +0000 |
| commit | a30b941ea73995e7a92bdc925b6267652a6b3ee2 (patch) | |
| tree | dfae96b15951eb9726275d414b65f7bb73c631bd | |
| parent | c7f08a11dfbc4a49f7f11a2a4a92f41bfa5ec622 (diff) | |
| download | src-a30b941ea73995e7a92bdc925b6267652a6b3ee2.tar.gz src-a30b941ea73995e7a92bdc925b6267652a6b3ee2.zip | |
Don't package /var/db/locate.database
... or install an empty file.
The locate database is a mutable working file, and unsuitable to be
managed by pkg(8). locate(1) copes gracefully with the absence of
this file. Running /etc/periodic/weekly/310.locate will create the
locate database with the correct ownership and permissions.
Having the locate.database file as part of the FreeBSD-runtime package
means it gets zeroed if ever that package gets updated, which is
excessive, given that updating packages will typically only change a
relatively small number of the _filenames_ on the system -- unlike the
file contents. Plus it always causes it to flag as modified when
verifying checksums, but /var/db/locate.database is expected to be
modified by the user.
MFC after: 1 week
Reviewed by: manu
Differential Revision: https://reviews.freebsd.org/D45384
(cherry picked from commit 4354163e5fe8e180f62b892cdc3e51e3d7e4706f)
| -rw-r--r-- | etc/Makefile | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/etc/Makefile b/etc/Makefile index 05632e8ce710..6d53b64a79eb 100644 --- a/etc/Makefile +++ b/etc/Makefile @@ -82,10 +82,6 @@ distribution: mail/aliases ${DESTDIR}/etc/aliases; \ fi .endif -.if ${MK_LOCATE} != "no" - ${INSTALL} -o nobody -g ${BINGRP} -m 644 -T "package=runtime"\ - /dev/null ${DESTDIR}/var/db/locate.database -.endif cd ${.CURDIR}/..; ${INSTALL} -o ${BINOWN} -g ${BINGRP} -m 444 \ -T "package=runtime" ${FREEBSD} ${DESTDIR}/ .if ${MK_BOOT} != "no" |