Re: [PATCH v5 17/36] qapi/common.py: move build_params into gen.py

2020-10-07 Thread Eduardo Habkost
On Wed, Oct 07, 2020 at 11:21:51AM +0200, Markus Armbruster wrote: > John Snow writes: > > > Including it in common.py creates a circular import dependency; schema > > relies on common, but common.build_params requires a type annotation > > from schema. To type this properly, it needs to be moved

Re: [PATCH v5 17/36] qapi/common.py: move build_params into gen.py

2020-10-07 Thread John Snow
On 10/7/20 5:21 AM, Markus Armbruster wrote: John Snow writes: Including it in common.py creates a circular import dependency; schema relies on common, but common.build_params requires a type annotation from schema. To type this properly, it needs to be moved outside the cycle. Signed-off-by:

Re: [PATCH v5 17/36] qapi/common.py: move build_params into gen.py

2020-10-07 Thread Markus Armbruster
John Snow writes: > Including it in common.py creates a circular import dependency; schema > relies on common, but common.build_params requires a type annotation > from schema. To type this properly, it needs to be moved outside the > cycle. > > Signed-off-by: John Snow > Reviewed-by: Eduardo Ha

[PATCH v5 17/36] qapi/common.py: move build_params into gen.py

2020-10-05 Thread John Snow
Including it in common.py creates a circular import dependency; schema relies on common, but common.build_params requires a type annotation from schema. To type this properly, it needs to be moved outside the cycle. Signed-off-by: John Snow Reviewed-by: Eduardo Habkost Reviewed-by: Cleber Rosa