[Libguestfs] [PATCH] file: Fix file command on /dev/VG/LV paths (RHBZ#582484).

Matthew Booth mbooth at redhat.com
Tue Jun 8 16:38:34 UTC 2010


On 08/06/10 16:07, Richard W.M. Jones wrote:
>> From ba39ced8804765705f4c61a92db0fddb8d672c7d Mon Sep 17 00:00:00 2001
> From: Richard Jones<rjones at redhat.com>
> Date: Tue, 8 Jun 2010 16:04:01 +0100
> Subject: [PATCH] file: Fix file command on /dev/VG/LV paths (RHBZ#582484).
>
> Previous commit 4df593496e116dfb635731c058b7627e81fc179c broke the
> "file" command on logical volume paths, since these are symbolic
> links.  We*should*  follow these (only).
>
> This inadvertantly broke virt-inspector too, which indicates that
> we need more regression testing in this area.  Since carrying whole
> Fedora images around could make the distribution even larger than
> now, I'm not sure at the moment how to do this.
>
> Thanks to Matt Booth for diagnosing this bug.
> ---
>   daemon/file.c    |    7 ++++++-
>   src/generator.ml |    2 +-
>   2 files changed, 7 insertions(+), 2 deletions(-)

ACK.

However, as discussed on IRC this call is now a bit of a mess. Could we 
please deprecate it in favour of a new one?

On a related note, perhaps we should update virt-inspector to use 
vfs_type instead of file.

Matt
-- 
Matthew Booth, RHCA, RHCSS
Red Hat Engineering, Virtualisation Team

M:       +44 (0)7977 267231
GPG ID:  D33C3490
GPG FPR: 3733 612D 2D05 5458 8A8A 1600 3441 EA19 D33C 3490




More information about the Libguestfs mailing list