[Cin] cinelerra dpx support patches

[Andrew Randrianasulu]

On Tuesday, April 5, 2022, Phyllis Smith <phylsmith2017 at gmail.com> wrote:

> Checked into GIT, Manuel's DPX list (*read but not write*) new files and
> modifications to existing files.
> Added in the changes Andrew provided in addition to accommodate other O/S
> like Termux (hopefully did not miss anything so if Andrew could check, that
> would be appreciated).
>

I'll wait a bit until second half of ogg ifdefing lands, so git hopefully
rebase with less rejects...

>
> I had already redone the libvpx.patch when it failed so I could compile,
> but later put in Andrew's 0070-Fix-libdpx-patch.patch anyway.  And of
> course, added in 0071-Add-without-libdpx-to-configure.patch and did a
> rebuild.
>

i think I missed piece adding LIBDPX among group of deps so libdpx's status
displayed at the end of configure

0059-Add-libdpx-status-to-configure-summary.patch

from megapile_3

>
> BUT, I have not been able to actually get a DPX list file to load (hoping
> Andrea can).
>

how you tried to construct said list? manually, by script, as output from
another program?





> BTW, was the 1 line fix in filelist.C an existing bug?
>

+++ b/cinelerra-5.1/cinelerra/filelist.C
@@ -115,7 +115,7 @@ int FileList::open_file(int rd, int wr)
                                int width = asset->width, height =
asset->height;
                                char string[BCTEXTLEN];
                                int len = strlen(list_prefix);
- int ret = fread(string, 1, strlen(list_prefix), stream);
+ int ret = fread(string, 1, len, stream);
                                fclose(stream);
                                result = len == ret ? 0 : 1;
                                if( !result && !strncasecmp(string,
list_prefix, len)) {


?

yeah, it seem to just finally use this 'len' variable as intended...



> Thanks for the auxiliary help. ...Phyllis
>
> On Sun, Apr 3, 2022 at 10:07 AM Andrew Randrianasulu <
> randrianasulu at gmail.com> wrote:
>
>> try my series?
>>
>>
>>
>> On Sunday, April 3, 2022, Phyllis Smith <phylsmith2017 at gmail.com> wrote:
>>
>>> I have been testing Manuel's Cinelerra DPXlist changes.  the last 2
>>> days, but have not yet gotten it to compile.  I will send his files/patches
>>> but it is included at:
>>>
>>> https://github.com/manuvi/cinelerra
>>>
>>> On Sun, Apr 3, 2022 at 8:53 AM Andrew Randrianasulu via Cin <
>>> cin at lists.cinelerra-gg.org> wrote:
>>>
>>>> hopefully "c++17" thing just cosmetical for now? I'll try to remove
>>>> this line...
>>>>
>>>>
>>>> https://github.com/manuvi/dpx/commit/48d8e8532244aebddd1f3ab
>>>> e7f30efe83cbd61f5
>>>>
>>>>
>>>>
>>>> On Saturday, March 5, 2022, Andrew Randrianasulu <
>>>> randrianasulu at gmail.com> wrote:
>>>>
>>>>> note: requires c++17 language support, so my gcc 5.5 out of action....
>>>>>
>>>>> ---------- Forwarded message ----------
>>>>> From: *Андрей Рандрианасулу* <randrik at mail.ru>
>>>>> Date: Saturday, March 5, 2022
>>>>> Subject: cinelerra dpx support patches
>>>>> To: randrianasulu <randrianasulu at gmail.com>
>>>>>
>>>>>
>>>>>
>>>>> not tested yet
>>>>> --
>>>>> Андрей  Рандрианасулу
>>>>>
>>>>> --
>>>> Cin mailing list
>>>> Cin at lists.cinelerra-gg.org
>>>> https://lists.cinelerra-gg.org/mailman/listinfo/cin
>>>>
>>>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.cinelerra-gg.org/pipermail/cin/attachments/20220405/942007da/attachment.htm>