[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 3/7] python/qemu: add README.rst
From: |
John Snow |
Subject: |
Re: [PATCH 3/7] python/qemu: add README.rst |
Date: |
Fri, 5 Jun 2020 12:18:24 -0400 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.5.0 |
On 6/5/20 10:56 AM, Vladimir Sementsov-Ogievskiy wrote:
> 03.06.2020 03:15, John Snow wrote:
>> Add a short readme that explains the package hierarchy, which will be
>> visible while browsing the source on e.g. gitlab/github.
>>
>> Signed-off-by: John Snow <jsnow@redhat.com>
>> ---
>> python/qemu/README.rst | 8 ++++++++
>> 1 file changed, 8 insertions(+)
>> create mode 100644 python/qemu/README.rst
>>
>> diff --git a/python/qemu/README.rst b/python/qemu/README.rst
>> new file mode 100644
>> index 00000000000..96447b90616
>> --- /dev/null
>> +++ b/python/qemu/README.rst
>> @@ -0,0 +1,8 @@
>> +python/qemu/
>> +------------
>> +
>> +This directory serves as the root of a `Python PEP 420 implicit
>> +namespace package <<https://www.python.org/dev/peps/pep-0420/>`_.
>> +
>> +Each directory below is assumed to be an installable Python module that
>
> As far as I understand terminology: s/module/package/
>
>> +will be available under the ``qemu.<module>`` namespace.
>>
>
> and here.
>
> Possibly I'm wrong. anyway:
> Reviewed-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
>
Oh, you're right. Each file is a module, each folder is a package. Thank
you for spotting that.
- [PATCH 0/7] python: create installable package, John Snow, 2020/06/02
- [PATCH 1/7] python/qemu: create qemu.lib module, John Snow, 2020/06/02
- [PATCH 3/7] python/qemu: add README.rst, John Snow, 2020/06/02
- [PATCH 2/7] python/qemu: formalize as package, John Snow, 2020/06/02
- [PATCH 6/7] python/qemu: Add flake8 to pipenv, John Snow, 2020/06/02
- [PATCH 7/7] python/qemu: add mypy to pipenv, John Snow, 2020/06/02
- [PATCH 4/7] python/qemu: Add pipenv support, John Snow, 2020/06/02