[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