Re: [PATCH v3 25/34] qapi: New special feature flag "deprecated"

[Markus Armbruster]

Philippe Mathieu-Daudé <address@hidden> writes:

> On 3/15/20 3:46 PM, Markus Armbruster wrote:
>> Unlike regular feature flags, the new special feature flag
>> "deprecated" is recognized by the QAPI generator.  For now, it's only
>> permitted with commands, events, and struct members.  It will be put
>> to use shortly.
>
> What about using a '@DeprecatedSince' tag?
> Ah, that would apply to the whole structure and not a particular member...
>
> I am wondering because sometimes we deprecate something for 2 releases
> but then once the deadline passed we forgot to remove it. Having the
> release version deadline along with deprecation would help automating
> removal checks.
> Maybe we can use a tag with release version, such 'deprecated_since:5.0'?

Intended consumer for "since <version>" information?

So far, it's been just developers.  Searching the QAPI schema for
deprecated stuff past its grace period hasn't been the problem;
remembering to search for it has been.