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.