<div dir="ltr"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><blockquote type="cite" style="margin:0px 0px 0px 0.8ex;border-left:2px solid rgb(114,159,207);padding-left:1ex"><blockquote type="cite" style="margin:0px 0px 0px 0.8ex;border-left:2px solid rgb(114,159,207);padding-left:1ex"><div dir="ltr"><div style="font-size:small">I need clarification please.</div><div style="font-size:small">1) Thirdparty-Makefile-giflib.patch that Mat attached is definitely needed and should be checked into GIT? probably yes?<br></div><div style="font-size:small">2) libaom-v3.2.0.patch1 that Andrew provided on Jan. 08; is this needed? probably not?<br></div><div style="font-size:small">3) have I missed some other changes? probably not?<br></div></div><br></blockquote></blockquote><br><div><snip></div><div>I tested both again to make sure.</div><div>=== giflib ===</div><div>Without the giflib change:</div><div>Makefile:</div><div>  giflib.cfg_params=echo "exec true" > ./configure; chmod +x ./configure;</div><div>log:</div><div>  CONFIGURING giflib</div><div>  cd giflib* &&   ./configure echo "exec true" > ./configure; chmod +x ./configure; </div><div>and the configure file is 0 bytes.</div><div><br></div><div>With the giflib change:</div><div>Makefile:</div><div>  giflib.cfg_vars=echo "exec true" > ./configure; chmod +x ./configure;</div><div>log:</div><div>  CONFIGURING giflib</div><div>  cd giflib* && echo "exec true" > ./configure; chmod +x ./configure;  ./configure  </div><div>and the configure file is 10 bytes.</div><div><br></div><div>So, it builds without the fix because the last cmd is a chmod which return OK.</div><div>But it is by accident. So the fix should be in the git.</div></div></blockquote><div><span class="gmail_default" style="font-size:small">OK, checked into GIT as stated after building on my laptop system.</span> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div><br></div><div>There are two more with bad configure files, I will look at them. The Makefile always</div><div>calls ./configure in the root of the unpacked source directory, so there should be </div><div>a configure script, which in case of unused, should return true.</div></div></blockquote><div><span class="gmail_default" style="font-size:small">OK, will wait for your feedback if more needed.</span> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div><br></div><div>== libaom patch====</div><div>The 20220108_Andrew patch for libaom, which changes __LINUX__ to __linux__ is not</div><div>needed for X86_64 builds, it builds fine with either.</div><div>However, it fails building on Debian_11/aarch64 compiling</div><div>libaom_v3.2.0/aom_ports/arm_cpudetect.c .</div><div><br></div><div>With the patch, which lowercases __linux__, it builds fine.</div><div><br></div><div>So this fix should be in the git too.</div></div></blockquote><div><span class="gmail_default" style="font-size:small"><span class="gmail_default" style="font-size:small">OK, checked into GIT as stated after building on my laptop system.</span> </span> </div></div></div>