On Sat, Nov 26, 2016 at 09:00:41PM +0000, Rostislav Pehlivanov wrote:
> On 26 October 2016 at 23:43, Rostislav Pehlivanov <atomnu...@gmail.com>
> wrote:
>
> >
> >
> > On 26 October 2016 at 23:33, Andreas Cadhalpun <
> > andreas.cadhal...@googlemail.com> wrote:
> >
> >> On 27.10.2016 00:16, Rostislav Pehlivanov wrote:
> >> > On 26 October 2016 at 22:48, James Almer <jamr...@gmail.com> wrote:
> >> >
> >> >> On 10/26/2016 6:19 PM, Rostislav Pehlivanov wrote:
> >> >>> Also removes url_feof from libavformat.v which should have been
> >> >>> removed long ago and changed the multipart jpeg boundary tag to
> >> >>> ffmpeg rather than ffserver (it's arbitrary).
> >> >>
> >> >> [...]
> >> >>
> >> >>> diff --git a/libavformat/libavformat.v b/libavformat/libavformat.v
> >> >>> index c961cd8..47d5ddc 100644
> >> >>> --- a/libavformat/libavformat.v
> >> >>> +++ b/libavformat/libavformat.v
> >> >>> @@ -1,19 +1,6 @@
> >> >>> LIBAVFORMAT_MAJOR {
> >> >>> global:
> >> >>> av*;
> >> >>> - #FIXME those are for ffserver
> >> >>> - ff_inet_aton;
> >> >>> - ff_socket_nonblock;
> >> >>> - ff_rtsp_parse_line;
> >> >>> - ff_rtp_get_local_rtp_port;
> >> >>> - ff_rtp_get_local_rtcp_port;
> >> >>> - ffio_open_dyn_packet_buf;
> >> >>> - ffio_set_buf_size;
> >> >>> - ffurl_close;
> >> >>> - ffurl_open;
> >> >>> - ffurl_write;
> >> >>> - #those are deprecated, remove on next bump
> >> >>> - url_feof;
> >> >>> local:
> >> >>> *;
> >> >>> };
> >> >>
> >> >> No, this can't be done until the next major bump. url_feof is even
> >> guarded
> >> >> by an scheduled FF_API define.
> >> >>
> >> >> The rest should be ok, but anything that implies a library ABI break
> >> can't
> >> >> be done just yet. Wait for other comments and/or testing. This removes
> >> a
> >> >> lot of things after all.
> >> >>
> >> > Fixed version attached without removing url_feof (apparently that's
> >> meant
> >> > to go in the next bump, wasn't forgotten)
> >>
> >> No, none of the symbols can be removed without a major bump. The simple
> >> reason
> >> is that a ffserver built from the 3.1 branch still has to work with the
> >> libavformat from the 3.2 branch.
> >>
> >> Best regards,
> >> Andreas
> >>
> >> _______________________________________________
> >> ffmpeg-devel mailing list
> >> ffmpeg-devel@ffmpeg.org
> >> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> >>
> >
> > Thanks for telling me,
> > Attached patch will only remove the ffserver program and documentation
> > relating to it.
> > The ffm demuxer/muxer will be removed with the next bump like url_feof.
> >
>
>
> Since a month has passed, reynaldo still hasn't responded, I think it's
> long overdue to push this patch. I've attached a new version of the patch
> which only removes the ffserver program (without touching anything else),
> which had been OK'd by James Almer.IIUC reynaldo is very busy. have you looked at ffserver ? I think helping reynaldo and updating ffserver so it doesnt violate the API would make alot more sense for FFmpeg than removing it. Its a long existing feature and tool. And i dont think its really that hard to update it, iam not the author of it iam not an expert on that code so i might be wrong or miss things but the big mess in it seems just moving parameters around between demuxers and muxers. and a few uses of network and rtp functions which should be used through some public API or made available through a new one everyone likes ... Dont you think its better if some people work together for a few days to update ffserver and have a cool feature more in FFmpeg, a cleaned up ffserver with some more modernized features. And the people liking ffserver be happy and thankfull vs. Discussions and eventually a vote about removing ffserver and then whatever the outcome some people "loosing" and being unhappy I know i should update ffserver myself if i care so much, but my todo is getting longer and longer (from new coverity issues that need checking to all the little things here and there like updating the git sync script earlier today so it deletes a mistakly added branch and doesnt spread it to one of our git mirrors at least ...) Above said, this mail is not intended to block or veto anyones patch [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Dictatorship: All citizens are under surveillance, all their steps and actions recorded, for the politicians to enforce control. Democracy: All politicians are under surveillance, all their steps and actions recorded, for the citizens to enforce control.
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
