[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] Re: [PATCH v2] qemu-img: Initial progress printing supp
Re: [Qemu-devel] Re: [PATCH v2] qemu-img: Initial progress printing support
Thu, 31 Mar 2011 12:49:30 +0100
On Thu, Mar 31, 2011 at 12:38 PM, Kevin Wolf <address@hidden> wrote:
> Am 31.03.2011 13:15, schrieb Jes Sorensen:
>> On 03/31/11 12:38, Kevin Wolf wrote:
>>> Am 30.03.2011 14:16, schrieb address@hidden:
>>>> From: Jes Sorensen <address@hidden>
>>>> This adds the basic infrastructure for supporting progress output
>>>> on the command line, as well as progress support for qemu-img commands
>>>> 'rebase' and 'convert'.
>>>> Signed-off-by: Jes Sorensen <address@hidden>
>>> Thanks, applied to the block branch.
>>> I think we should consider turning the progress output on by default if
>>> qemu-img is run from a terminal.
>> I have been a little reluctant to do this because it will break the ABI
>> for tools running qemu-img from a GUI etc.
> That's the reason for the "from a terminal" part. If we check for
> isatty(), we should handle these cases just fine.
Yes, I think checking for a tty is enough precaution and allows users
to get the benefit of the progress bar. TBH I'd probably forget to
add -p half the time :).