On Tue, Jun 10, 2014 at 03:18:36PM +0200, Pino Toscano wrote:

--- daemon/lvm-filter.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/daemon/lvm-filter.c b/daemon/lvm-filter.c index ed10a48..6976bab 100644 --- a/daemon/lvm-filter.c +++ b/daemon/lvm-filter.c @@ -200,7 +200,7 @@ vgchange (const char *vgchange_flag) CLEANUP_FREE char *err = NULL; int r = command (NULL, &err, str_lvm, "vgchange", vgchange_flag, NULL); if (r == -1) { - reply_with_error ("vgchange: %s", err); + reply_with_error ("vgchange %s: %s", vgchange_flag, err); return -1; }

ACK. Improving error messages is basically something we can always do, so I think sensible error message improvements shouldn't need to be acked in future. Thanks, Rich. -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones Read my programming and virtualization blog: http://rwmj.wordpress.com virt-builder quickly builds VMs from scratch http://libguestfs.org/virt-builder.1.html