|
| From: | Markus Armbruster |
| Subject: | Re: [PATCH 0/2] qapi/qom: use correct field name when getting/setting alias properties |
| Date: | Thu, 22 Jul 2021 15:25:53 +0200 |
| User-agent: | Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux) |
Since patch submitters tend to submit code that works for the success
case, I like to test a few failure cases before anything else. Gotcha:
$ qemu-system-x86_64 -machine pc,pflash0=xxx qemu-system-x86_64:
Property 'cfi.pflash01.drive' can't find value 'xxx'
The error message is misleading.
This is not a "must not commit" issue. Fixing a regression in time for
the release at the price of a bad error message is still a win. The bad
error message needs fixing all the same, just not necessarily before the
release.
Since mere thinking doesn't rock the release boat: any ideas on how this
could be fixed?
| [Prev in Thread] | Current Thread | [Next in Thread] |