<div dir="ltr"><div class="gmail_default" style="font-size:small">This OpenEXR package is a "bag of worms" -- weird, hard, and worse package to integrate in Cinelerra. GG has taken your patches, made quite a few changes, and got it to work at 2.4.1. Tomorrow he will test to see how it does on the 18+ distros to see if it will work for the monthly builds. Meanwhile, I think he has come to the conclusion that it is not worthwhile to upgrade it any more because there do not seem to be any gains in doing so -- there releases seem to be only code cleanup but not no features whatsoever.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Will see if it works for the monthly builds, and then probably never again will be upgraded. gg/phyllis<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Mar 18, 2020 at 9:35 AM Andrew Randrianasulu <<a href="mailto:randrianasulu@gmail.com">randrianasulu@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
<br>
В сообщении от Wednesday 18 March 2020 18:00:06 Good Guy написал(а):<br>
> These changes look good. Thanks.<br>
> and now the inevitable "but"..., these are just suggestions,<br>
> but they are based on my past experiences.<br>
> <br>
> 1) don't leave dead code in the product unless you think it may be useful<br>
> in the future.<br>
<br>
Well, removed ilmbase from both <a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a> and thirparty/Makefile<br>
<br>
If you decide -stdc+11 requirement is NOT show stopper, then you can try to apply those<br>
and two other patches, also attached this time, because without them nothing will work<br>
<br>
put openexr-2.4.1 patches into thirdparty/src, put 2.4.1 tarball there, too (you may want to repack it,<br>
or I can find xz version ...):<br>
<br>
<a href="https://sources.voidlinux.org/openexr-2.4.1/" rel="noreferrer" target="_blank">https://sources.voidlinux.org/openexr-2.4.1/</a><br>
<br>
sha256sum openexr-2.4.1.tar.gz<br>
3ebbe9a8e67edb4a25890b98c598e9fe23b10f96d1416d6a3ff0732e99d001c1 openexr-2.4.1.tar.gz<br>
<br>
matches with sha256sum provideed at this page:<br>
<a href="https://www.freshports.org/graphics/openexr" rel="noreferrer" target="_blank">https://www.freshports.org/graphics/openexr</a><br>
<br>
Try on oldest system first? (due to c++11 req. it may fail first).<br>
<br>
> examples: the configure script is already sort of trashy, since it is the<br>
> driver for a set<br>
> of old products. The main reason the thirdparty design exists is that you<br>
> can't trust the<br>
> maintainers to not break your build. PLEASE be aware that every month any<br>
> mod you<br>
> make has to work on 18 platforms that cover a span of 10 years or so. Any<br>
> break that<br>
> is historical can stop several builds. The build system is a xen host<br>
> running an<br>
> embedded busybox system that only operates the build. No servers, no<br>
> installed extras.<br>
> It operates 3 sets of build iterators. If any of them fail, more passes<br>
> (4,5,...). Each pass<br>
> is risky, since it is almost always some kind of emergency patch, and<br>
> usually<br>
> downgrades the results. Another example, /usr/X11R7/lib has not been<br>
> around for a<br>
> while on most platforms. It also as permuted variants /usr/lib/X11<br>
> /usr/share/X11/ etc.<br>
> If the change is really needed for a particular platform, then it may need<br>
> to be "specialized".<br>
> <br>
> 2) I am not sure, but it looks like there is a new version of OpenEXR<br>
> (2.4.1). I tried it,<br>
> but it requires the a very recent version of cmake. Currently, there seems<br>
> to be a struggle<br>
> to replace autotools with cmake,meson,waf ... Meson broke many builds. I<br>
> rewrote the<br>
> dav1d meson build as a makefile, and tried to give it to the developers,<br>
> but they declined to<br>
> carry it. Every time they tweak the build, I have to mod the makefile.<br>
> Not nice. And now<br>
> the suggestion, If you are going to refactor the OpenEXR build, please see<br>
> if it is possbile<br>
> to get it to work with the more recent version. It is not essential to<br>
> make it work, since it<br>
> is already working with a usable version, but keeping up with 40 or so<br>
> libraries is a major<br>
> task, and maintenance is continuous. Try to make sure that the work you do<br>
> will be good<br>
> for as long as possible.<br>
> <br>
> 3) I personally wish to thank you (and all of you) who have helped to<br>
> improve the result.<br>
> It has always been a curiosity that almost all of the technical help is not<br>
> from any "techy"<br>
> component, (eg. google, apple, adobe, ...) but from real users that are<br>
> just plain interested<br>
> in having something nice to use. It is highly appreciated for your work on<br>
> a large section<br>
> of source code/libraries. Thank you.<br>
> <br>
> gg<br>
> PS. The documentation rework is just outstanding.<br>
> <br>
> <br>
> <br>
> <br>
> <br>
> On Wed, Mar 18, 2020 at 6:07 AM Andrew Randrianasulu <<br>
> <a href="mailto:randrianasulu@gmail.com" target="_blank">randrianasulu@gmail.com</a>> wrote:<br>
> <br>
> > Only <a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a> patch this time ..<br>
> ><br>
> > diff --git a/cinelerra-5.1/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a> b/cinelerra-5.1/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a><br>
> > index f10af67..7375fa9 100644<br>
> > --- a/cinelerra-5.1/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a><br>
> > +++ b/cinelerra-5.1/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a><br>
> > @@ -8,7 +8,7 @@ AC_LANG_CPLUSPLUS<br>
> > AC_LANG_C<br>
> > AC_PROG_CXX<br>
> ><br>
> > -CFG_CFLAGS+=" -fno-omit-frame-pointer -fno-math-errno -fno-signed-zeros"<br>
> > +CFG_CFLAGS+=" -std=c++11 -fno-omit-frame-pointer -fno-math-errno<br>
> > -fno-signed-zeros"<br>
> > CFG_CFLAGS+=" -pthread -Wall"<br>
> > # misguided pedantic warnings<br>
> > # this is extra work, not a gain...<br>
> > @@ -239,14 +239,14 @@ PKG_3RD([giflib],[yes],<br>
> > libutil.a ],<br>
> > [ . ])<br>
> ><br>
> > -PKG_3RD([ilmbase],[auto],<br>
> > - [ilmbase-2.2.1],<br>
> > - [ Iex/.libs/libIex.a \<br>
> > - IexMath/.libs/libIexMath.a \<br>
> > - Half/.libs/libHalf.a \<br>
> > - Imath/.libs/libImath.a \<br>
> > - IlmThread/.libs/libIlmThread.a ],<br>
> > - [ Iex Half Imath config IlmThread ])<br>
> > +#PKG_3RD([ilmbase],[auto],<br>
> > +# [ilmbase-2.2.1],<br>
> > +# [ Iex/.libs/libIex.a \<br>
> > +# IexMath/.libs/libIexMath.a \<br>
> > +# Half/.libs/libHalf.a \<br>
> > +# Imath/.libs/libImath.a \<br>
> > +# IlmThread/.libs/libIlmThread.a ],<br>
> > +# [ Iex Half Imath config IlmThread ])<br>
> ><br>
> > PKG_DEF([ladspa], [ladspa-0.4.17], [], [])<br>
> ><br>
> > @@ -337,11 +337,16 @@ PKG_3RD([mjpegtools],[yes],<br>
> > mpeg2enc/.libs/libmpeg2encpp.a ],<br>
> > [ . lavtools utils ])<br>
> ><br>
> > -PKG_3RD([openexr],[auto],<br>
> > - [openexr-2.2.1],<br>
> > - [ IlmImf/.libs/libIlmImf.a \<br>
> > - IlmImfUtil/.libs/libIlmImfUtil.a ],<br>
> > - [ IlmImf config ])<br>
> > +PKG_3RD([openexr],[yes],<br>
> > + [openexr-2.4.1],<br>
> > + [ IlmBase/Iex/.libs/libIex.a \<br>
> > + IlmBase/IexMath/.libs/libIexMath.a \<br>
> > + IlmBase/Half/.libs/libHalf.a \<br>
> > + IlmBase/Imath/.libs/libImath.a \<br>
> > + IlmBase/IlmThread/.libs/libIlmThread.a<br>
> > + OpenEXR/IlmImf/.libs/libIlmImf.a \<br>
> > + OpenEXR/IlmImfUtil/.libs/libIlmImfUtil.a ],<br>
> > + [ Iex Half Imath IlmThread IlmImf config usr/include/OpenEXR ])<br>
> ><br>
> > ---<br>
> ><br>
> > I force-disabled openEXR probe so it will NOT pick up system headers/libs,<br>
> > discovered new includes demand -std=c++11 in CFLAGS :}<br>
> ><br>
> > seems to work ....<br>
> > --<br>
> > Cin mailing list<br>
> > <a href="mailto:Cin@lists.cinelerra-gg.org" target="_blank">Cin@lists.cinelerra-gg.org</a><br>
> > <a href="https://lists.cinelerra-gg.org/mailman/listinfo/cin" rel="noreferrer" target="_blank">https://lists.cinelerra-gg.org/mailman/listinfo/cin</a><br>
> ><br>
> <br>
<br>
<br>
-- <br>
Cin mailing list<br>
<a href="mailto:Cin@lists.cinelerra-gg.org" target="_blank">Cin@lists.cinelerra-gg.org</a><br>
<a href="https://lists.cinelerra-gg.org/mailman/listinfo/cin" rel="noreferrer" target="_blank">https://lists.cinelerra-gg.org/mailman/listinfo/cin</a><br>
</blockquote></div>