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

Petr Vorel pvorel@suse.cz
Fri Jan 14 18:44:05 CET 2022


Hi Joerg,

> Hi

> Am 1/14/2022 um 3:23 PM schrieb Cyril Hrubis:
> > 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.
Cyril, maybe we should have kept the way to completely disable building metadata
until we actually use it (i.e. until runtest files are replaced).

OTOH this way give the change to (embedded) distros to fix host build setup
issues :). I was testing Buildroot last night and it has problems with build on
archs which have cross-compile flags (CFLAGS) incompatible with host flags
(HOST_CFLAGS). This might require deeper changes to introduce LTP host build
(Buildroot support host autotools builds, but LTP is not fully autootools
project - we use only autoconf).

> 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.
Hard to help without seeing logs.

Kind regards,
Petr

> 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) :)

> Joerg


More information about the ltp mailing list