Re: [Qemu-devel] [PATCH] Document --rtc-td-hack, --localtime and --startdate as deprecated

[Paolo Bonzini <pbonzini@redhat.com>]

On 20/02/2018 16:05, Paolo Bonzini wrote:
> On 20/02/2018 16:01, Thomas Huth wrote:
>> These options have been marked in a comment in qemu-options.hx as
>> deprecated in 2009 already (see commit 1ed2fc1fa35fadc0d6), but we
>> never informed the users about these deprecations. Let's catch up
>> on that omission now.
>>
>> Signed-off-by: Thomas Huth <thuth@redhat.com>
>> ---
>>  qemu-doc.texi | 13 +++++++++++++
>>  vl.c          |  5 +++++
>>  2 files changed, 18 insertions(+)
>>
>> diff --git a/qemu-doc.texi b/qemu-doc.texi
>> index 2f19980..7494042 100644
>> --- a/qemu-doc.texi
>> +++ b/qemu-doc.texi
>> @@ -2736,6 +2736,19 @@ filesystem test suite. Also it requires the CAP_DAC_READ_SEARCH capability,
>>  which is not the recommended way to run QEMU. This backend should not be
>>  used and it will be removed with no replacement.
>>  
>> +@subsection -rtc-td-hack (since 2.12.0)
>> +
>> +The @code{--rtc-td-hack} option has been replaced by
>> +@code{--rtc driftfix=slew}.
>> +
>> +@subsection -localtime (since 2.12.0)
>> +
>> +The @code{--localtime} option has been replaced by @code{--rtc localtime}.
>> +
>> +@subsection -startdate (since 2.12.0)
>> +
>> +The @code{--startdate} option has been replaced by @code{--rtc @var{date}}.
>> +
>>  @section qemu-img command line arguments
>>  
>>  @subsection convert -s (since 2.0.0)
>> diff --git a/vl.c b/vl.c
>> index 0feaca7..84121bf 100644
>> --- a/vl.c
>> +++ b/vl.c
>> @@ -3417,6 +3417,8 @@ int main(int argc, char **argv, char **envp)
>>                  break;
>>              case QEMU_OPTION_localtime:
>>                  rtc_utc = 0;
>> +                warn_report("This option is deprecated, "
>> +                            "use '--rtc localtime' instead.");
>>                  break;
>>              case QEMU_OPTION_vga:
>>                  vga_model = optarg;
>> @@ -3672,6 +3674,8 @@ int main(int argc, char **argv, char **envp)
>>                  };
>>  
>>                  qdev_prop_register_global(&slew_lost_ticks);
>> +                warn_report("This option is deprecated, "
>> +                            "use '--rtc driftfix=slew' instead.");
>>                  break;
>>              }
>>              case QEMU_OPTION_acpitable:
>> @@ -3876,6 +3880,7 @@ int main(int argc, char **argv, char **envp)
>>                   */
>>                  break;
>>              case QEMU_OPTION_startdate:
>> +                warn_report("This option is deprecated, use '--rtc' instead.");
>>                  configure_rtc_date_offset(optarg, 1);
>>                  break;
>>              case QEMU_OPTION_rtc:
> 
> Queued, thanks.

... except qemu_rtc_opts has no implied_opt_name, soo it's "-rtc base".
I fixed that up.

Paolo