Re: PR/59838 CVS commit: src/usr.sbin/mtree

["Jose Luis Duran via gnats" <gnats-admin%NetBSD.org@localhost>]

The following reply was made to PR bin/59838; it has been noted by GNATS.

From: Jose Luis Duran <jlduran%gmail.com@localhost>
To: gnats-bugs%netbsd.org@localhost
Cc: gnats-admin%netbsd.org@localhost, netbsd-bugs%netbsd.org@localhost
Subject: Re: PR/59838 CVS commit: src/usr.sbin/mtree
Date: Fri, 19 Dec 2025 11:46:29 -0300

 >  > We do we need to check F_TYPE twice? It is in the outer if...
 >
 >  The outer check is for the case where "/set" should not be printed at
 >  all, without any key/value pair, for example:
 >      $ mtree -cn -k flags -R flags,type -p /usr/bin/ -x
 >  In that corner case, no "/set" line should be printed.
 
 Forgot to mention my doubt:
 Reading the comment: "Note that we always output a /set record for the
 first directory.", my fear is that it could produce an invalid
 specification, because there would be no "/set" record, or, in the
 previous case, just an empty "/set" record with no key/value pairs.