[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v7] block/raw-posix.c: Fixes raw_getlength() on
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v7] block/raw-posix.c: Fixes raw_getlength() on Mac OS X so that it reports the correct length of a real CD |
Date: |
Mon, 19 Jan 2015 19:11:43 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.3 (gnu/linux) |
Programmingkid <address@hidden> writes:
> On Jan 16, 2015, at 3:22 AM, Markus Armbruster wrote:
>
>> Programmingkid <address@hidden> writes:
>>
>>> This patch allows Mac OS X to use a real CDROM disc in
>>> QEMU. Testing this patch will require using QEMU v2.2.0 because the
>>> current git version has a bug in it that prevents /dev/cdrom from
>>> being used. "make check" did pass and my Debian boot disc did work.
>>>
>>> Signed-off-by: John Arbuckle <address@hidden>
>>
>> Subject line is too long, and the body lacks line breaks. Suggest
>> something like:
>>
>> block/raw-posix.c: Fix raw_getlength() on Mac OS X for CD
>
> Unfortunately I can't change the subject line after posting the first
> version of the patch. Others have to know the history of this patch. I
> will try to include the file name in the future.
Oh yes, you can!
If you care about patch history, do this:
block/raw-posix.c: Fix raw_getlength() on Mac OS X for CD
This patch allows Mac OS X to use a real CDROM disc in QEMU.
Testing this patch will require using QEMU v2.2.0 because the
current git version has a bug in it that prevents /dev/cdrom from
being used. "make check" did pass and my Debian boot disc did work.
Signed-off-by: John Arbuckle <address@hidden>
---
v7:
* Totally rewritten.
* Subject changed from "block/raw-posix.c: Fixes raw_getlength() on
Mac OS X so that it reports the correct length of a real CD"
v6:
[more history, if you care...]
A good commit message is worth a little inconvenience.
>> This patch allows Mac OS X to use a real CDROM disc in QEMU.
>> Testing this patch will require using QEMU v2.2.0 because the
>> current git version has a bug in it that prevents /dev/cdrom from
>> being used. "make check" did pass and my Debian boot disc did work.
>
> Ok. I just don't know when to break it. I guess I will stick to pinky length.
I recommend to limit line length to 70 characters.