[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 06/12] vl: Factor configure_blockdev() out of
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v2 06/12] vl: Factor configure_blockdev() out of main() |
Date: |
Thu, 07 Mar 2019 20:08:15 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux) |
Eric Blake <address@hidden> writes:
> On 3/7/19 11:23 AM, Markus Armbruster wrote:
>> Signed-off-by: Markus Armbruster <address@hidden>
>>
>> # Conflicts:
>> # vl.c
>
> How'd you get git to preserve the leading #? Generally, I find conflicts
> details useful for cherry-picked backports, but pointless for rebased
> patches intended as original upstream material. And git defaults to
> stripping lines with leading # when composing a commit message.
I've messed up too many commit message by having fill-paragraph flow a #
to the beginning of a line, so I added
[commit]
cleanup = scissors
to my .gitconfig. I've been messing up commit messages with leftover
crap ever since, but leftover crap has proven less confusing to my
reviewers than missing lines.
> May be
> worth cleaning up before the actual pull request.
Certainly.
git-commit(1):
--cleanup=<mode>
This option determines how the supplied commit message should be
cleaned up before committing. The <mode> can be strip, whitespace,
verbatim, scissors or default.
strip
Strip leading and trailing empty lines, trailing whitespace,
commentary and collapse consecutive empty lines.
whitespace
Same as strip except #commentary is not removed.
verbatim
Do not change the message at all.
scissors
Same as whitespace except that everything from (and including)
the line found below is truncated, if the message is to be
edited. "#" can be customized with core.commentChar.
# ------------------------ >8 ------------------------
default
Same as strip if the message is to be edited. Otherwise
whitespace.
The default can be changed by the commit.cleanup configuration
variable (see git-config(1)).
- [Qemu-devel] [PATCH v2 00/12] pc: Support firmware configuration with -blockdev, Markus Armbruster, 2019/03/07
- [Qemu-devel] [PATCH v2 04/12] sysbus: Fix latent bug with onboard devices, Markus Armbruster, 2019/03/07
- [Qemu-devel] [PATCH v2 02/12] qom: Move compat_props machinery from qdev to QOM, Markus Armbruster, 2019/03/07
- [Qemu-devel] [PATCH v2 01/12] qdev: Fix latent bug with compat_props and onboard devices, Markus Armbruster, 2019/03/07
- [Qemu-devel] [PATCH v2 03/12] vl: Fix latent bug with -global and onboard devices, Markus Armbruster, 2019/03/07
- [Qemu-devel] [PATCH v2 06/12] vl: Factor configure_blockdev() out of main(), Markus Armbruster, 2019/03/07
- [Qemu-devel] [PATCH v2 11/12] pc: Support firmware configuration with -blockdev, Markus Armbruster, 2019/03/07
- [Qemu-devel] [PATCH v2 10/12] pc_sysfw: Pass PCMachineState to pc_system_firmware_init(), Markus Armbruster, 2019/03/07
- [Qemu-devel] [PATCH v2 12/12] docs/interop/firmware.json: Prefer -machine to if=pflash, Markus Armbruster, 2019/03/07
- [Qemu-devel] [PATCH v2 05/12] vl: Improve legibility of BlockdevOptions queue, Markus Armbruster, 2019/03/07