Re: pkg/45044: archivers/libarchive minix support

[Joerg Sonnenberger <joerg%britannica.bec.de@localhost>]

On Fri, Jun 10, 2011 at 03:20:01PM +0000, tcort%minix3.org@localhost wrote:
> Minix UIDs are 16-bits and Minix GIDs are 8-bits. The tar format
> supports much larger UIDs/GIDs. libarchive should handle the case where
> the UIDs/GIDs overflow the archive_entry uid/gid fields by setting
> the uid/gid to something valid (example: the uid of the nobody user and
> the gid of the nobody group) and it should produce a warning when doing so.

I don't think the complexity of this is justified with the gain.
Ultimately, you are either ending up ignoring the UID/GID or it won't
work at all. The behavior is already "set it to something valid", since
it truncates. Doing a user name look up in this piece of code is
completely inacceptable. In short, I feel this is a case of "use saner
types or just cope". You can, of course, bring this up on the libarchive
list or bug tracker, but I am going to repeat my answer in that case.

Joerg