[FFmpeg-devel] [PATCH 7/7] avformat: add Changelog entry for librist and bump minor

[Paul B Mahol]

On Fri, Mar 12, 2021 at 12:21 AM Marton Balint <cus at passwd.hu> wrote:

>
>
> On Sun, 7 Mar 2021, Marton Balint wrote:
>
> >
> >
> > On Sun, 7 Mar 2021, Paul B Mahol wrote:
> >
> >> On Sun, Mar 7, 2021 at 9:02 AM Marton Balint <cus at passwd.hu> wrote:
> >>
> >>>
> >>>
> >>> On Sun, 7 Mar 2021, Paul B Mahol wrote:
> >>>
> >>> > On Sun, Mar 7, 2021 at 1:57 AM James Almer <jamrial at gmail.com>
> wrote:
> >>> >
> >>> >> On 3/6/2021 9:51 PM, Paul B Mahol wrote:
> >>> >> > On Sun, Mar 7, 2021 at 1:35 AM Marton Balint <cus at passwd.hu>
> wrote:
> >>> >> >
> >>> >> >>
> >>> >> >>
> >>> >> >> On Sun, 7 Mar 2021, Paul B Mahol wrote:
> >>> >> >>
> >>> >> >>> How you tested this?
> >>> >> >>
> >>> >> >> with the librist tools.
> >>> >> >>
> >>> >> >>>
> >>> >> >>> And why you have not asked nicely before taking working on this?
> >>> >> >>
> >>> >> >> You wrote you have given up. It is your behaviour which is not
> very
> >>> >> nice.
> >>> >> >> After your continuous ignorance of my requests, I try to help
> your
> >>> work
> >>> >> >> get merged, and this is your feedback...
> >>> >> >>
> >>> >> >
> >>> >> >
> >>> >> > Why are you so aggressive? Other devs raised issues that looks not
> >>> >> resolved.
> >>> >>
> >>> >> I don't see aggressiveness from his part. I see you making things
> >>> >> incredibly more complex than they should be.
> >>> >>
> >>> >
> >>> > The code in functional part is not much different from initial
> versions.
> >>> >
> >>> > So I really doubt that fifo filling up issue have been properly
> >>> addressed.
> >>>
> >>> I can take a look at that if you provide command lines to reproduce.
> >>>
> >>
> >> I couldn't reproduce it, so you have to ask others.
> >
> > I found one issue in librist code which can be related and might cause
> > ffmpeg not to recover from a fifo fillup. I have reported it here:
> >
> > https://code.videolan.org/rist/librist/-/issues/93
>
> I'd love to see rist support in 4.4, so maybe we should apply this series
> as is and fix any remaining issues later...
>


Not gonna block it.



>
> Regards,
> Marton
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".