Bug 270657
| Summary: | [NEW PORT] sysutils/plocate - locate that can find all files | ||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | Ports & Packages | Reporter: | sec | ||||||||||
| Component: | Individual Port(s) | Assignee: | Robert Clausecker <fuz> | ||||||||||
| Status: | Closed FIXED | ||||||||||||
| Severity: | Affects Only Me | CC: | fuz | ||||||||||
| Priority: | --- | ||||||||||||
| Version: | Latest | ||||||||||||
| Hardware: | Any | ||||||||||||
| OS: | Any | ||||||||||||
| Attachments: |
|
||||||||||||
Thank you for your submission. Please check the following issues:
- please prefer DISTVERSION over PORTVERSION if possible
- do not use REINPLACE_CMD for static replacements. Instead, supply patch
files. Use the "make makepatch" mechanism to generate these (see § 4.4
Porter's Handbook)
- please use /var/db instead of ${PREFIX}/var/db for database files.
- avoid not printing out shell commands in the Makefile
- including a header file directly from FILESDIR is unconventional but ok.
Rest looks ok. Will proceed with a build test once you address these issues.
Please make sure to set "maintainer-approval" on your patches so they are more likely to be looked at.
Created attachment 241965 [details]
Updated port
Addressed your comments: - use DISTVERSION - added patch files & removed use of REINPLACE_CMD - moved database to /var/db - copied compat c file & header to WORKDIR during compilation Thank you for your updated submission. Here are some additional issues I found - your patch generates a directory plocate in the root of the ports tree. The port needs to be in a subdirectory of its category, e.g. in directory sysutils/plocate. - Make sure to hook up your new port in sysutils/Makefile - You request that a group plocate be generated. Please add an entry for this group to the GIDs file in the root of the ports collection. - LICENSE should be GPLv2+ as README has the “or any later version” clause Port looks good otherwise and builds fine! Will likely be able to commit once these three issues are addressed. Created attachment 242160 [details]
Updates new port patch
Adressed the comments:
+ changed License
+ added port to sysutils/Makefile
+ added GID allocation
Looking better! I found two additional issues in my extended testing: - your port misses USES=pkgconfig. Without it, it can't find libzstd. - your port stages a stray a CACHEDIR.TAG file. Either patch the build scripts so the file is no longer staged or add it to pkg-plist: ====> Checking for pkg-plist issues (check-plist) ===> Parsing plist ===> Checking for items in STAGEDIR missing from pkg-plist Error: Orphaned: /var/db/plocate/CACHEDIR.TAG ===> Checking for items in pkg-plist which are not in STAGEDIR ===> Error: Plist issues found. *** Error code 1 Created attachment 242198 [details]
Updated new port patch
Added CACHEDIR.TAG to plist
Added pkgconfig to USES
Thanks. Will commit with my next batch. A commit in branch main references this bug: URL: https://cgit.FreeBSD.org/ports/commit/?id=1a556fa3ec8dbea28a7b34b2e9e2f036b0276ad6 commit 1a556fa3ec8dbea28a7b34b2e9e2f036b0276ad6 Author: Stefan `Sec` Zehl <sec@42.org> AuthorDate: 2023-05-14 08:26:17 +0000 Commit: Robert Clausecker <fuz@FreeBSD.org> CommitDate: 2023-05-18 08:19:45 +0000 sysutils/plocate: Fast & privacy-respecting locate utility plocate is a locate(1) based on posting lists, completely replacing mlocate with a much faster (and smaller) index. It is suitable as a default locate on your system. Like mlocate and slocate, the returned file set is user-dependent, ie. a user will only see a file if find(1) would list it (all directories from the root have +rx permissions). WWW: https://plocate.sesse.net/ PR: 270657 GIDs | 2 +- sysutils/Makefile | 1 + sysutils/plocate/Makefile (new) | 34 ++++ sysutils/plocate/distinfo (new) | 3 + sysutils/plocate/files/315.plocate.in (new) | 32 ++++ sysutils/plocate/files/mntent.h (new) | 62 ++++++++ sysutils/plocate/files/mntent_compat.c++ (new) | 177 +++++++++++++++++++++ sysutils/plocate/files/patch-conf.cpp (new) | 11 ++ .../plocate/files/patch-io__uring__engine.h (new) | 10 ++ sysutils/plocate/files/patch-meson.build (new) | 56 +++++++ sysutils/plocate/files/patch-updatedb.cpp (new) | 11 ++ sysutils/plocate/files/updatedb.conf.sample (new) | 1 + sysutils/plocate/pkg-descr (new) | 5 + sysutils/plocate/pkg-plist (new) | 9 ++ 14 files changed, 413 insertions(+), 1 deletion(-) Thank you for your submission. |
Created attachment 241313 [details] new port patch This is a new port of plocate. It is a modern locate that indexes all files but only shows the ones you have read access for.