[x264-devel] [PATCH 2/4] configure: replace usage of [[ with POSIX-compatible [
Alexander Strasser
eclipse7 at gmx.net
Thu Jan 16 20:37:54 CET 2020
On 2020-01-16 18:07 +0100, Diego Biurrun wrote:
> On 14/01/2020 18.18, Ethan Sommer wrote:
> > --- a/configure
> > +++ b/configure
> > @@ -1020,7 +1017,7 @@ if [ $compiler = GNU ]; then
> > CPU_ENDIAN="big-endian"
> > - elif !(${cross_prefix}strings -a conftest.o | grep -q EGIB && ${cross_prefix}strings -a conftest.o | grep -q naidnePF) ; then
> > + elif ! (${cross_prefix}strings -a conftest.o | grep -q EGIB && ${cross_prefix}strings -a conftest.o | grep -q naidnePF) ; then
> > die "endian test failed"
> > @@ -1611,7 +1608,8 @@ cat conftest.log >> config.log
> > [ "$SRCPATH" != "." ] && ln -sf ${SRCPATH}/Makefile ./Makefile
> > -mkdir -p common/{aarch64,arm,mips,ppc,x86} encoder extras filters/video input output tools
> > +mkdir -p common/aarch64 common/arm common/mips common/ppc common/x86 \
> > + encoder extras filters/video input output tools
>
> (Random) unrelated changes, please leave these out.
Those changes on their own are good and required for conformance to
POSIX shell.
Sneaking them in under this very specfific commit message is of course
not ok.
Alexander
More information about the x264-devel
mailing list