[FFmpeg-devel] [PATCH] Remove the ffserver program and the ffm muxer/demuxer

[wm4]

On Sun, 27 Nov 2016 14:05:44 +0100
Michael Niedermayer <michael at niedermayer.cc> wrote:

> On Sat, Nov 26, 2016 at 09:00:41PM +0000, Rostislav Pehlivanov wrote:
> > On 26 October 2016 at 23:43, Rostislav Pehlivanov <atomnuker at gmail.com>
> > wrote:
> >   
> > >
> > >
> > > On 26 October 2016 at 23:33, Andreas Cadhalpun <  
> > > andreas.cadhalpun at googlemail.com> wrote:  
> > >  
> > >> On 27.10.2016 00:16, Rostislav Pehlivanov wrote:  
> > >> > On 26 October 2016 at 22:48, James Almer <jamrial at 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 at 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.

Where does this come from? It seems never ever it happened that someone
wanted to actually update and maintain ffserver. That's why we want to
remove it. The reason for removal isn't going to change apparently.
Basically nobody wants to touch it, and those who tried apparently
failed. It's over.