Bug 252558 - misc/findutils update to 4.8.0
Summary: misc/findutils update to 4.8.0
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Rainer Hurling
URL: https://lists.gnu.org/archive/html/bu...
Keywords: buildisok, needs-qa
Depends on:
Blocks:
 
Reported: 2021-01-10 14:45 UTC by Klaus Aehlig
Modified: 2021-01-11 21:17 UTC (History)
2 users (show)

See Also:


Attachments
patch for misc/findutils to update to 4.8.0 (1.22 KB, patch)
2021-01-10 14:45 UTC, Klaus Aehlig
aehlig: maintainer-approval+
Details | Diff
Modified with ordered vars, corrected patch file, and without sig file (4.38 KB, patch)
2021-01-11 12:26 UTC, Rainer Hurling
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Klaus Aehlig 2021-01-10 14:45:42 UTC
Created attachment 221440 [details]
patch for misc/findutils to update to 4.8.0

Update misc/findutils to 4.8.0

Release notes: https://lists.gnu.org/archive/html/bug-findutils/2021-01/msg00044.html
Comment 1 Automation User 2021-01-10 15:13:07 UTC
Build and package info is available at https://gitlab.com/swills/freebsd-ports/pipelines/239622951
Comment 2 Fernando Apesteguía freebsd_committer 2021-01-11 08:28:00 UTC
^Triage: If there is a changelog or release notes URL available for this version, please add it to the URL field.

Q/A: 
 /tmp/252558/misc/findutils/files/patch-configure: patch was not generated using ``make makepatch''.  It is recommended to use ``make makepatch'' when you need to [re-]generate a patch to ensure proper patch format.
 /tmp/252558/misc/findutils/files/patch-gnulib-lib-fdopendir.c: patch was not generated using ``make makepatch''.  It is recommended to use ``make makepatch'' when you need to [re-]generate a patch to ensure proper patch format.

Thanks!
Comment 3 Rainer Hurling freebsd_committer 2021-01-11 12:26:22 UTC
Created attachment 221455 [details]
Modified with ordered vars, corrected patch file, and without sig file

Hi Klaus,

Thanks for patch.

After investigating a bit, I came across a few points with your patch that I would like to clarify:

- 'portlint -AC' indicated that something is wrong with files/patch-gl_lib_fdopendir.c. It turned out that this file was not created by 'make makepatch' and was also wrongly named.

- 'portclippy Makefile' (from ports-mgmt/portfmt) gives hints about the order of variables in the Makefile. I have reordered accordingly.

- It is not clear to me what the signature file is downloaded for. Surely it is not used afterwards? For now, I commented it out. If you agree, it will be removed from the Makefile and the distinfo.

- LOCALSTATDIR should point to the location where glocate finds the locate.database? This does not seem to work. glocate does not use the existing locate.database under /var/db until this is explicitly specified in the call:

glocate --database=/var/db/locate.database xyz...
Comment 4 Klaus Aehlig 2021-01-11 19:56:18 UTC
(In reply to Rainer Hurling from comment #3)

Thanks for resorting the variables and refreshing/renaming the patches. I didn't touch them, as they did not need any modifications and had had this name for years; but I'm perfectly happy using this opportunity to clean things up.

> It is not clear to me what the signature file is downloaded for.

The signature is indeed not used for building. It is used (manually, at the moment) to verify that the archive really originates from the correct upstream. As this, however, is purely a maintainer task, you're right, there is no value in fetching the signature as part of a regular build, given that the archive is verified against the hashes in distinfo.

> LOCALSTATDIR should point to the location where glocate finds the locate.database? This does not seem to work. glocate does not use the existing locate.database under /var/db until this is explicitly specified in the call: [...]

LOCALSTATEDIR points indeed to the directory(!) where gupdatedb generates its database and glocate takes the database from (unless, of course, told by options to use a different database). However, glocate calls its database locatedb and not locate.database. This difference in names can be seen as a bug or as a feature, depending on whether one wants to reuse locate's database or completely independent coexistence of locate and glocate. This port always took the latter route.
Comment 5 Rainer Hurling freebsd_committer 2021-01-11 20:38:23 UTC
(In reply to Klaus Aehlig from comment #4)

Thanks for the reply. And thanks for clarification, now I also understand the LOCALSTATEDIR var enough ;)


After removing the signature issue completely, I prepared an updated patch for a review on Phabricator [1]. My mentors will review and it is likely to get there approval in the next couple of days :) 

[1] https://reviews.freebsd.org/D28101
Comment 6 commit-hook freebsd_committer 2021-01-11 21:16:20 UTC
A commit references this bug:

Author: rhurlin
Date: Mon Jan 11 21:15:40 UTC 2021
New revision: 561272
URL: https://svnweb.freebsd.org/changeset/ports/561272

Log:
  misc/findutils update to 4.8.0

  Changelog: https://lists.gnu.org/archive/html/bug-findutils/2021-01/msg00044.html

  PR:		252558
  Submitted by:	Klaus Aehlig <aehlig@linta.de> (maintainer)
  Approved by:	tcberner (mentor)
  Differential Revision:	https://reviews.freebsd.org/D28101

Changes:
  head/misc/findutils/Makefile
  head/misc/findutils/distinfo
  head/misc/findutils/files/patch-configure
  head/misc/findutils/files/patch-gl_lib_fdopendir.c
  head/misc/findutils/files/patch-gnulib-lib-fdopendir.c
  head/misc/findutils/pkg-descr
Comment 7 Rainer Hurling freebsd_committer 2021-01-11 21:17:31 UTC
Committed, thanks :)