[Cin] libaom 3.2.0 patch1 fixed for arm/linux?

[mat]

On Thu, 2022-01-13 at 14:38 +0300, Andrew Randrianasulu wrote:
> 
> 
> On Wednesday, January 12, 2022, Phyllis Smith
> <phylsmith2017 at gmail.com> wrote:
> > MatN and Andrew,
> > I need clarification please.
> > 1) Thirdparty-Makefile-giflib.patch that Mat attached is definitely
> > needed and should be checked into GIT? probably yes?
> > 2) libaom-v3.2.0.patch1 that Andrew provided on Jan. 08; is this
> > needed? probably not?
> > 3) have I missed some other changes? probably not?
> > 
> 
> I can't comment much on libgif patch (Cingg worked dor me without
> it), but my patch only should have effect on Linux/arm, I do not have
> suitable chroot for testing, but from stackoverflow answers it seems
> __linux__ is right define, even if ANDROID and TERMUX are all-caps,
> and *BSD spelled with capitalization too, see example:
> https://www.boost.org/doc/libs/1_63_0/boost/config/platform/bsd.hpp) 

<snip>
I tested both again to make sure.
=== giflib ===
Without the giflib change:
Makefile:
giflib.cfg_params=echo "exec true" > ./configure; chmod +x ./configure;
log:
CONFIGURING giflib
cd giflib* && ./configure echo "exec true" > ./configure; chmod +x
./configure; 
and the configure file is 0 bytes.

With the giflib change:
Makefile:
giflib.cfg_vars=echo "exec true" > ./configure; chmod +x ./configure;
log:
CONFIGURING giflib
cd giflib* && echo "exec true" > ./configure; chmod +x ./configure;
./configure 
and the configure file is 10 bytes.

So, it builds without the fix because the last cmd is a chmod which
return OK.
But it is by accident. So the fix should be in the git.

There are two more with bad configure files, I will look at them. The
Makefile always
calls ./configure in the root of the unpacked source directory, so
there should be 
a configure script, which in case of unused, should return true.

== libaom patch====
The 20220108_Andrew patch for libaom, which changes __LINUX__ to
__linux__ is not
needed for X86_64 builds, it builds fine with either.
However, it fails building on Debian_11/aarch64 compiling
libaom_v3.2.0/aom_ports/arm_cpudetect.c .

With the patch, which lowercases __linux__, it builds fine.

So this fix should be in the git too.

MatN
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.cinelerra-gg.org/pipermail/cin/attachments/20220113/70756189/attachment-0001.htm>