[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 4/6] qemu-option: clean up id vs. list->merge_lists
From: |
Markus Armbruster |
Subject: |
Re: [PATCH v2 4/6] qemu-option: clean up id vs. list->merge_lists |
Date: |
Tue, 10 Nov 2020 10:54:11 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.1 (gnu/linux) |
Paolo Bonzini <pbonzini@redhat.com> writes:
> On 10/11/20 09:29, Markus Armbruster wrote:
>> As we will see below, there are exceptions where we reject
>> options that used to work. Do we want that?
>
> I think that, as long as we gain in consistency, we do. The special
> casing of "id" is a wart of the current parser and the less it shows
> that "id" is treated specially, the best.
>
> Deprecating or downright rejecting working combinations is always
> walking a fine line, but here we're almost in https://xkcd.com/1172/
> territory.
I find it hard to disagree ;)
- Re: [PATCH v2 3/6] qemu-option: restrict qemu_opts_set to merge-lists QemuOpts, (continued)
[PATCH v2 4/6] qemu-option: clean up id vs. list->merge_lists, Paolo Bonzini, 2020/11/09
- Re: [PATCH v2 4/6] qemu-option: clean up id vs. list->merge_lists, Markus Armbruster, 2020/11/09
- Re: [PATCH v2 4/6] qemu-option: clean up id vs. list->merge_lists, Paolo Bonzini, 2020/11/09
- Re: [PATCH v2 4/6] qemu-option: clean up id vs. list->merge_lists, Markus Armbruster, 2020/11/09
- Re: [PATCH v2 4/6] qemu-option: clean up id vs. list->merge_lists, Paolo Bonzini, 2020/11/09
- Re: [PATCH v2 4/6] qemu-option: clean up id vs. list->merge_lists, Markus Armbruster, 2020/11/10
- Re: [PATCH v2 4/6] qemu-option: clean up id vs. list->merge_lists, Paolo Bonzini, 2020/11/10
- Re: [PATCH v2 4/6] qemu-option: clean up id vs. list->merge_lists,
Markus Armbruster <=
[PATCH v2 5/6] qemu-option: move help handling to get_opt_name_value, Paolo Bonzini, 2020/11/09
[PATCH v2 6/6] qemu-option: warn for short-form boolean options, Paolo Bonzini, 2020/11/09
Re: [PATCH v2 for-5.2 0/6] Deprecate or forbid crazy QemuOpts cases, no-reply, 2020/11/09