On 16/11/2020 16:33, Ilya Maximets wrote: > On 11/16/20 5:28 PM, Stokes, Ian wrote: >>> On Wed, 11 Nov 2020 04:25:30 -0500 >>> Mark Gray <[email protected]> wrote: >>> >>>> ovs/dirs.py should be auto-generated using the template >>>> ovs/dirs.py.template at build time. This will set the >>>> ovs.dirs python variables with a value specified by the >>>> environment or, if the environment variable is not set, from >>>> the build system. >>>> >>>> Signed-off-by: Mark Gray <[email protected]> >>> >>> Acked-By: Timothy Redaelli <[email protected]> >>> >> >> Thanks Timothy, I've pushed this series to master now. > > Thanks, Ian. But we might need a follow up on this change. > I think, the main reason why dirs.py existed is that we need > it while doing setup of python package, i.e. we should have > it during execution of python/ovs/setup.py. Setup script > already checks if version.py exists, and we might want check > for dirs.py too, otherwise python package will be non-functional.
Sounds small enough. I will update this. > > Also, gitignore file seems to have dir.py and not dirs.py. Probably why it got checked in the first time. > > Best regards, Ilya Maximets. > _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
