Re: [libvirt] [PATCH v3 02/22] build-aux: rewrite augest test generator in Python

On Wed, Sep 25, 2019 at 03:25:39PM +0200, Ján Tomko wrote: > On Tue, Sep 24, 2019 at 03:58:43PM +0100, Daniel P. Berrangé wrote: > > As part of an goal to eliminate Perl from libvirt build tools, > > rewrite the augeas-gentest.pl tool in Python. > > > > This was a straight conversion, manually going line-by-line to > > change the syntax from Perl to Python. Thus the overall structure > > of the file and approach is the same. > > > > The use of $(AUG_GENTEST) as a dependancy in the makefiles needed > > s/dependancy/dependency/ > > > to be fixed, because this was assumed to be the filename of the > > script, but is in fact a full shell command line. > > > > This is the case regardless of the Perl->Python conversion > and can be done upfront to reduce the churn in this patch. > Introduced by commit fb59cf7a5824b9c876737dcbf6aac97c29b1444a > > > Signed-off-by: Daniel P. Berrangé <berrange(a)redhat.com&gt; > > --- > > Makefile.am | 2 +- > > build-aux/augeas-gentest.pl | 60 --------------------------- > > build-aux/augeas-gentest.py | 72 +++++++++++++++++++++++++++++++++ > > src/Makefile.am | 3 +- > > src/bhyve/Makefile.inc.am | 4 +- > > src/interface/Makefile.inc.am | 2 +- > > src/libxl/Makefile.inc.am | 4 +- > > src/locking/Makefile.inc.am | 6 +-- > > src/logging/Makefile.inc.am | 2 +- > > src/lxc/Makefile.inc.am | 4 +- > > src/network/Makefile.inc.am | 2 +- > > src/node_device/Makefile.inc.am | 2 +- > > src/nwfilter/Makefile.inc.am | 2 +- > > src/qemu/Makefile.inc.am | 4 +- > > src/remote/Makefile.inc.am | 4 +- > > src/secret/Makefile.inc.am | 2 +- > > src/storage/Makefile.inc.am | 2 +- > > src/vbox/Makefile.inc.am | 2 +- > > src/vz/Makefile.inc.am | 2 +- > > 19 files changed, 97 insertions(+), 84 deletions(-) > > delete mode 100755 build-aux/augeas-gentest.pl > > create mode 100755 build-aux/augeas-gentest.py > > Since this is a new file with clean history, it might actually deserve > a better location than build-aux and we can leave this directory to > Automake and gnulib to do whatever magic they do there. > Also note that the directory is in .gitignore. (I added the exception > for .pl files back when I added files here) > > Would 'scripts' be too vague? Could be a good place to put the helper > scripts for generating QEMU caps files since I never seem to remember > its name and tests/ is growing quite big. That's a good question. As you see from this series, we've got random scripts scattered all over the sub-dirs. build-aux/ was in some sense to avoid polluting the top level dir. If we create 'scripts/' should we put everything in there, or just stuff that's related to the top level, and keep everything else in their current subdirs ?

Regards, Daniel