[LTP] [RFC PATCH v3 1/5] travis: Add build script and use it in travis

Petr Vorel pvorel@suse.cz
Mon Dec 4 21:49:49 CET 2017


Hi Cyril,

thanks for your time!

> I find this output to be a bit too verbose, frankly I do not remeber
> checking it content of the config.log ever.
It is verbose. The reason I added it is, that we can check that travis is doing what we
want (e.g. header / library is really installed / not installed).
I didn't realize, that include/config.h would be enough and it's much smaller.
But I suppose you don't want any output in case of successful configure run, do you?

> > +    make $make_opts
> > +    make $make_opts DESTDIR="$PREFIX" SKIP_IDCHECK=1 install
>    ^
>    Here you use spaces instead of tabs.
Thanks. Copy & paste issue.

> > +}
> > +
> > +build_in_tree()
> > +{
> > +	echo "=== autotools ==="
> > +	make autotools
> > +
> > +	echo "=== configure ==="
> > +	if ! ./configure $CONFIGURE_OPTS_IN_TREE $@; then
> > +		echo "== ERROR: configure failed, config.log =="
> > +		cat config.log
> > +		exit 1
> > +	fi
> > +
> > +	echo "== config.log =="
> > +	cat config.log

> Here as well.

> > +# vim: set ft=sh ts=4 sts=4 sw=4 noet:

> I prefer not to clobber source code with this kind of editor hints.
OK.


Kind regards,
Petr


More information about the ltp mailing list