23 Oct
2017
23 Oct
'17
11:57 a.m.
Hi Neels,
On Sun, Oct 22, 2017 at 09:34:07PM +0200, Neels Hofmeyr wrote:
This patch was merged, but I frankly don't like
what it does.
Building in a separate dir is fine, but it does so with a bunch of changes
along that introduce indenting errors and potentially destroy the scripts'
certainty of building cleanly every time.
I though there was two subsequent runs of "autoreconf -fi", one for building
in-source
and one for building out-of-source, and the rm-rf is the cause of it. Maybe I have
misunderstood what the original code was intending to do. In any case, an out-of-source
build "autoreconf -fi" will not have different results from one for in-source,
which is why I thought the patch is a good idea.
A common scripts for three lines seems rather odd to me, too - but then,
I don't really care all too much.
I'm not opposed to reverting it. Your call.
--
- Harald Welte <laforge(a)gnumonks.org> http://laforge.gnumonks.org/
============================================================================
"Privacy in residential applications is a desirable marketing option."
(ETSI EN 300 175-7 Ch. A6)