[libvirt PATCH 246/351] meson: src: add check-augeas test
Pavel Hrdina
phrdina at redhat.com
Tue Jul 28 08:48:27 UTC 2020
On Tue, Jul 28, 2020 at 10:22:27AM +0200, Peter Krempa wrote:
> On Tue, Jul 28, 2020 at 10:09:11 +0200, Pavel Hrdina wrote:
> > On Tue, Jul 28, 2020 at 09:51:40AM +0200, Peter Krempa wrote:
> > > On Thu, Jul 16, 2020 at 11:58:02 +0200, Pavel Hrdina wrote:
> > > > Signed-off-by: Pavel Hrdina <phrdina at redhat.com>
> > > > ---
> > > > scripts/check-augeas.sh | 12 ++++++++++++
> > > > scripts/meson.build | 1 +
> > > > src/Makefile.am | 17 -----------------
> > > > src/meson.build | 18 +++++++++++++++++-
> > > > 4 files changed, 30 insertions(+), 18 deletions(-)
> > > > create mode 100644 scripts/check-augeas.sh
> > >
> > > [...]
> > >
> > > > @@ -857,3 +860,16 @@ test(
> > > > args: [ check_aclrules_prog.path(), files('remote/remote_protocol.x'), stateful_driver_source_files ],
> > > > env: runutf8,
> > > > )
> > > > +
> > > > +if augparse_prog.found()
> > >
> > > Can't we use foreach on augeas_test_data and invoke the test
> > > individually for each file rather than adding a script which does the
> > > same?
> >
> > Sure we can, I wanted to stay as close as to autotools where
> > check-augeas is a single target. Using foreach would result into having
> > separate target for each file. In general it is most likely better but
> > I was trying to avoid any changes like this with the rewrite and do them
> > as a followup patches to cleanup the build system.
>
> In autotools the loop is in the automake file (yes, still shell),
> while here you put it into a separate script file. I'd argue that
> putting the loop into the meson file (regardless of how many test
> targets it creates) is more equivalent to the original source than this
> way.
>
> Given that you are now changing all the shell helpers to python I'd
> rather see this done in pure meson, than having a python file looper
> intermediate helper.
The change to python is already done. But I don't care so I can redo it.
Pavel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20200728/f0ea3cf7/attachment-0001.sig>
More information about the libvir-list
mailing list