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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxx>
> > ---
> > 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
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux