[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v5 10/20] docs/interop: Convert qemu-ga-ref to rST
From: |
Markus Armbruster |
Subject: |
Re: [PATCH v5 10/20] docs/interop: Convert qemu-ga-ref to rST |
Date: |
Tue, 22 Sep 2020 16:13:10 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.1 (gnu/linux) |
Peter Maydell <peter.maydell@linaro.org> writes:
> On Tue, 22 Sep 2020 at 13:01, Markus Armbruster <armbru@redhat.com> wrote:
>>
>> Peter Maydell <peter.maydell@linaro.org> writes:
>> > The Sphinx interop manual has the usual footer:
>> > https://www.qemu.org/docs/master/interop/index.html
>> > "©2020, The QEMU Project Developers."
>> >
>> > The system manual has an explicit "License" section:
>> > https://www.qemu.org/docs/master/system/license.html
>> > but that's documenting the license of the program, not the
>> > manual (it's docs/system/license.rst).
>> >
>> > We could do any or all of:
>> > * decide that we're happy with the current situation
>> > * expand the "copyright" footer to something like
>> > '©2020, The QEMU Project Developers; this manual is GPLv2'
>> > * have a docs/foo/license.rst for each manual, and expand
>> > it to mention the documentation license as well as the
>> > code license
>> >
>> > Given that the Texinfo generated QMP/QGA references don't
>> > actually present this text to the reader, my inclination
>> > is to say that this is something we should address in
>> > a separate patchseries, not as part of this conversion.
>>
>> I think the manual should have a proper copyright notice.
>
> It does -- that's the standard Sphinx footer that reads
> "©2020, The QEMU Project Developers." What it's missing
> is more precise licensing information.
Yes.
>> If you'd prefer to (re-)add it later, stick in TODO comment.
>
> Where would you want a TODO to be ?
Before the patch, the licensing information is in
docs/interop/qemu-ga-ref.texi. That file gets replaced by
docs/interop/qemu-ga-ref.rst, losing the licensing information. What
about putting the TODO right there?