On 20.12.23 16:43, Peter Eisentraut wrote:
On 20.12.23 12:40, Andres Freund wrote:
Hm, or perhaps we should just get rid of sed use altogether. The
sepgsql case
is trivially translateable to perl, and postprocess_dtrace.sed isn't
much harder.
Maybe yeah, but also it seems fine as is and we can easily fix the
present issue ...
OTOH, I actually don't think it's valid to not have sed when you have
dtrace. Erroring out in a weird way in such an artificially
constructed test
doesn't really seem like a problem.
Agreed. So let's just make it not-required, and that should work.
Updated patch set attached.
I have committed these two.