On Wed, Oct 08, 2025 at 11:47:05AM +0200, Jan Beulich wrote: > Even the 2018 edition of The Open Group Base Specifications Issue 7 [1] > doesn't name -E as a standard option; only Issue 8 [2] does. As there's > nothing "extended" about the expression used, simply drop the -E. > > [1] https://pubs.opengroup.org/onlinepubs/9699919799/ > [2] https://pubs.opengroup.org/onlinepubs/9799919799/ > > Fixes: cb50e4033717 ("test/pdx: add PDX compression unit tests") > Signed-off-by: Jan Beulich <[email protected]>
Reviewed-by: Roger Pau Monné <[email protected]> I think the -E was carried over from a previous expression, that was indeed extended. > --- > In principle the -e could be dropped too, for being redundant. > > Hitting the problem with an older sed pointed out another problem here as > well: The failed invocation left a 0-byte pdx.h, which upon re-invocation > of make was (obviously) deemed up-to-date, thus causing the build to fail > again (until the bad file was actually removed). Hm, we could do something like: sed -e '/^#[[:space:]]*include/d' <$< >$@ || $(RM) $@ Thanks, Roger.
