diff options
author | Colin Watson <cjwatson@debian.org> | 2022-09-25 01:12:01 +0100 |
---|---|---|
committer | Colin Watson <cjwatson@debian.org> | 2022-09-25 01:12:01 +0100 |
commit | 8fe549986ccefff52762933c5a6a9238abfa0ea2 (patch) | |
tree | c46dadb5bbf3a4c81155daeea4fa6d530c1ec8cc | |
parent | a3d05c4894030ad1f2212b76ce4ec85857c1e64b (diff) |
Don't skip adding a database entry due to SO_MAN
An existing `SO_MAN` entry shouldn't stop us considering a page for
insertion into the database, because until we run `ult_src` we don't yet
know whether that page will be `ULT_MAN` (which should replace the
existing entry) or `SO_MAN` (which shouldn't).
* src/check_mandirs.c (test_manfile): Don't return early due to an
existing matching `SO_MAN` entry, but only due to an `ULT_MAN` entry.
* NEWS.md: Document this.
-rw-r--r-- | NEWS.md | 2 | ||||
-rw-r--r-- | src/check_mandirs.c | 2 |
2 files changed, 3 insertions, 1 deletions
@@ -13,6 +13,8 @@ Fixes: the correct type. * Store links in the database using the section and extension of the link rather than of the ultimate source file. + * Only skip adding a page to the database due to an existing entry if the + entry represents an ultimate source. Improvements: diff --git a/src/check_mandirs.c b/src/check_mandirs.c index 855898f5..a5046678 100644 --- a/src/check_mandirs.c +++ b/src/check_mandirs.c @@ -208,7 +208,7 @@ void test_manfile (MYDBM_FILE dbf, const char *file, const char *path) if (strcmp (exists->comp, info->comp ? info->comp : "-") == 0) { if (timespec_cmp (exists->mtime, info->mtime) == 0 && - exists->id < WHATIS_MAN) { + exists->id == ULT_MAN) { free_mandata_struct (exists); free_mandata_struct (info); return; |