Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 9 Sep 2013 19:51:22 -0400
From:      christos@zoulas.com (Christos Zoulas)
To:        =?utf-8?Q?Dag-Erling_Sm=C3=B8rgrav?= <des@des.no>
Cc:        FreeBSD Current <freebsd-current@freebsd.org>, d@delphij.net, Brooks Davis <brooks@FreeBSD.org>, Tim Kientzle <kientzle@acm.org>
Subject:   Re: [PATCH] mtree should not output size if the file is not a regular file
Message-ID:  <20130909235122.4A9C997129@rebar.astron.com>
In-Reply-To: <86zjrleg4h.fsf@nine.des.no> from =?utf-8?Q?Dag-Erling_Sm=C3=B8rgrav?= (Sep 10,  1:21am)

next in thread | previous in thread | raw e-mail | index | archive | help
On Sep 10,  1:21am, des@des.no (=?utf-8?Q?Dag-Erling_Sm=C3=B8rgrav?=) wrote:
-- Subject: Re: [PATCH] mtree should not output size if the file is not a reg

| Roll a large tarball (e.g. a complete FreeBSD installation).  Copy it to
| different machines with different filesystems.  Untar and run mtree on
| the result.  Notice that you get different output on each machine
| because they report different sizes for directories; one might report
| the actual on-disk size (which might vary depending on past contents)
| while the other might report the number of entries.

Yes, I agree. I would like to note that the current NetBSD code looks like:

        if (keys & F_SIZE &&
	    (flavor != F_NETBSD6 || S_ISREG(p->fts_statp->st_mode)))

which means that F_NETBSD6 did not print this, and we recently changed
it to print the size for compatibility with F_FREEBSD9... We also made
the default F_MTREE format to print the size. So I guess the thing to
do is change the code to:

        if (keys & F_SIZE &&
	    (flavor == F_FREEBSD9 || S_ISREG(p->fts_statp->st_mode)))

christos



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20130909235122.4A9C997129>