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 <[email protected]> http://laforge.gnumonks.org/ ============================================================================ "Privacy in residential applications is a desirable marketing option." (ETSI EN 300 175-7 Ch. A6)
