qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Max Reitz <mreitz@redhat.com>
To: MagicCat Software <magiccatsoftware@gmail.com>
Cc: Peter Maydell <peter.maydell@linaro.org>,
	qemu-devel qemu-devel <qemu-devel@nongnu.org>
Subject: Re: [Qemu-devel] Mount image file feature
Date: Sat, 29 Aug 2015 21:52:27 +0200	[thread overview]
Message-ID: <55E20D7B.7000208@redhat.com> (raw)
In-Reply-To: <94F6A4AC-9CB6-41A0-BD31-1E680C0700C0@gmail.com>

[-- Attachment #1: Type: text/plain, Size: 1325 bytes --]

On 29.08.2015 20:34, MagicCat Software wrote:
> 
> On Aug 29, 2015, at 2:01 PM, Max Reitz wrote:

[snip]

>> User-friendliness is always expensive,
>> difficult to maintain, and a neverending source of complaints.
> 
> Really? It has been months since Peter Maydell implemented my GUI patches
> for the cocoa interface, and I haven't seen a complaint about it yet.

I just looked into it, and without having an OS X to test it on I have a
few things to complain about:

(1) 'change' always uses 'raw' as the format, but the selector
apparently offers you the full range of image formats
(supportedImageFileTypes).

(2) I think it should be possible to add block devices with removable
media at runtime (I didn't try, though), and also remove them. The
current code doesn't account for that.

(3) supportedImageFileTypes is static. The list might change at any time
and the likelihood of people adding or removing formats there when
adding/removing them to/from the block layer itself is pretty low. Also,
the list depends on compile and (in the future at least) maybe even
runtime options. For instance, it's missing vdi, vhd (vpc), and vhdx.
vhdx in turn can be disabled at compile time.

(4) NSBeep()? Is that necessary? Won't the alert itself be audible if so
desired by the user?

Max


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 473 bytes --]

  parent reply	other threads:[~2015-08-29 19:52 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-08-27  1:05 [Qemu-devel] Mount image file feature G 3
2015-08-29 15:40 ` Max Reitz
2015-08-29 15:57   ` Programmingkid
2015-08-29 16:39     ` Max Reitz
2015-08-29 17:36       ` Programmingkid
2015-08-29 18:01         ` Max Reitz
2015-08-29 18:34           ` MagicCat Software
2015-08-29 19:34             ` Max Reitz
2015-08-29 20:06               ` Programmingkid
2015-08-29 23:10               ` Eric Blake
2015-08-30  0:02                 ` Programmingkid
2015-08-30  7:15                   ` Paolo Bonzini
2015-08-29 19:52             ` Max Reitz [this message]
2015-08-29 20:18               ` Programmingkid
2015-08-29 23:12                 ` Eric Blake
2015-08-29 23:31                   ` Programmingkid
2015-08-29 23:04         ` Eric Blake
2015-08-30  0:03           ` MagicCat Software
2015-08-31  8:02             ` Markus Armbruster
2015-08-30  6:47         ` Paolo Bonzini
2015-08-31  3:47           ` Programmingkid
2015-08-31  7:52         ` Markus Armbruster
2015-08-31 13:12           ` Programmingkid
2015-08-31 16:26             ` Markus Armbruster
2015-08-31 18:29               ` Programmingkid
2015-08-30  6:42       ` Paolo Bonzini
2015-08-31 20:13       ` Programmingkid
2015-08-31 20:26         ` Max Reitz
2015-08-31 20:33           ` Programmingkid
2015-09-02 14:31             ` Max Reitz
2015-09-02 14:38               ` Programmingkid
2015-09-03  9:46                 ` Markus Armbruster
2015-09-03 14:24                   ` Programmingkid
2015-09-03 16:26                     ` Markus Armbruster
2015-09-03 16:51                       ` Programmingkid
2015-09-03  9:34               ` Markus Armbruster

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=55E20D7B.7000208@redhat.com \
    --to=mreitz@redhat.com \
    --cc=magiccatsoftware@gmail.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).