On 8/2/23 10:27, Robin Jarry wrote:

Hi, Robin.  Thanks for the fix!

> The python/build folder contents are completely unrelated to the ovs
> python bindings. These files are only used during the build for various
> subsystems (docs, man pages, code generation, etc.).> 
> Having that folder in that location prevents from running:
> 
>       cd python && python3 -m build
> 
> Which is a way to generate PEP517 compatible source archives and binary
> wheel packages.
> 
> Move that folder into build-aux. Update PYTHONPATH accordingly.


ovstest library also not really related to the ovs python bindings,
but it is in the python folder as well.  So, it doesn't seem like
a best reason for a move.  Also, having a build module in build-aux
looks a bit repetitive.

Can we just re-name the folder to something like 'build_helpers'
and keep it in the current place instead?

Best regards, Ilya Maximets.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to