[LTP] [PATCH 1/1] configure.ac: Fix summary for disabled metadata

Cyril Hrubis chrubis@suse.cz
Fri Jan 14 15:51:57 CET 2022


Hi!
> > The JSON metadata file is going to replace the runtest files some day,
> > at least that is the longterm plan. Also the parser that extracts the
> > metadata from the sources is rather fast, compared to the LTP build and
> > it's self contained. There is really no reason to have a switch to
> > disable the metadat extraction step.
> > 
> > The documentation build, on the other hand, is rather slow and requires
> > asciidoc, which is the reason why there is a switch that can be used to
> > disable that. The only problem here is that the name is a bit confusing
> > now.
> > 
> Actually there is, that is where my interest in this flag comes from. We
> don't use runtest files (at least not for execution), so we don't need
> it and we had problems in out cross build environment, that were not
> easily fixed. From looking at the source I am not sure what the problem
> was and cannot spot it from looking at the source. But I know, that it
> wasn't just fixed by setting the correct HOSTCC and HOST_CFLAGS. I have
> a missing library in my head, but I can't see any.

That is really strange, there are practically no dependencies for the
metadata extractor. All that you have to have is a C compiler and libc.

> All I know is that I gave up fixing the build and I don't give up easily
> normally (a little easier if the functionality is not needed at all) :)

You should have reported that on the list. It's going to be really
cornerstone of the way how tests are executed so the build should get
fixed anyways.

I guess that we can add a flag that disables the metadata extraction
step, with a big red warning that describes the consequencies of the
choice.

-- 
Cyril Hrubis
chrubis@suse.cz


More information about the ltp mailing list