Xin LI <delp...@gmail.com> wrote:

> I have took a look at the change history, it seems that the find
> operation was introduced in r245265
> <https://svnweb.freebsd.org/base?view=revision&revision=245265> (brooks@,
> to support packaged base) and sort was initially implemented as find -s
> in r289451
> <https://svnweb.freebsd.org/base?view=revision&revision=289451> (ngie@,
> to make METALOG reproducible) then as sort in r328958
> <https://svnweb.freebsd.org/base?view=revision&revision=328958> (imp@,
> for portability).
> 
> I wonder if we could drop the sort and replace ${TZS} in line 100 with
> ${TZS:O} instead?

Makes sense to me.
 
> By the way, looking at
> https://github.com/freebsd/pkg/blob/master/libpkg/metalog.c , I wonder if
> the sort should really happen in pkg(8) instead?

Currently the METALOG is also used when creating the tarballs so sorting
only in pkg would be insufficient as long as tarballs are still supported.

Fabian
_______________________________________________
freebsd-current@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"

Reply via email to