[FFmpeg-devel] Documentation: ffserver.conf files

[Piero Bugoni]

--- Piero Bugoni <crboca32 at yahoo.com> wrote:

> Thanks, Victor, I will correct the things you
> mention
> below.
> 
> How do I submit an "svn diff" when there is no
> previous file to diff against?
> 

The answer to this was posted, but I still don't get
it. When I figure it out, I will submit as "svn diff".
for this go-round, it is still a tarfile.


> > 
> > In general:
> > a) I'm not sure if the usual FFmpeg license should
> > appear in 
> > configuration files.

Don't Know. Will Do if necessary.


> > b) The files should be submitted as an "svn diff"
> > patch.
> > 

They will be, after this review.

> > I feel also missing a patch to ffserver-doc.texi
> to
> > mention there are several
> > example html/ffserver.conf files available.

Will do next. I needed preliminary review of the conf
files first.


> > 
> > 
> > On ffserver.conf-1cam-flv:
> > 
> > ># ffserver and ffmpeg run on same machine with 
> > ># apache webserver.   
> > 
> > 
> > Trailing spaces, there are more cases in this file
> > and the others.
> > Apache, here and in the other files.
> > grep '  ' ffserver.conf* shows a few cases of two
> > spaces 
> > (not counting the legitimate needed for aligning),
> > instead of one.
> > 

Done.


> > >VideoBufferSize 32         # Adjust in
> conjunction
> > with q and bitrate ...
> > 
> > The hash could be aligned with the others.
> > 

Done.

> > ># Works with both firefox and IE. 
> > 
> > I'd use "Firefox and Internet Explorer", here and
> in
> > the other files.
> > 
Done



> > ># make sure to match the settings in this file
> with
> > those on the command line.
> > 
> > Make
> > 

Done.

> > >VideoSize 352x288           # cif size
> > 
> > CIF
> > 

Done.

> > ># Flash Player has a limit of 16,500 frames
> > ># when using a browser refresh the viewing 
> > ># page periodically
> > 
> > I would say:
> > 
> > # Flash Player has a limit of 16,500 frames.
> > # When using a browser please refresh the viewing 
> > # page periodically.

Edited accordingly.

> > 
> > ># How does adjusting these options from defaults
> > affect stream/quality 
> > ># IntraOnly
> > ># VideoGOPSize
> > 
> > I find this sentence confusing (I'm not a native
> > English speaker, though)
> > Maybe you should add:
> > 
> > # FIXME
> > # Explain how does adjusting these options from
> > defaults affect stream/quality.
> > # IntraOnly
> > # VideoGOPSize
> > 

Done.

> > This also appears in some other of your conf
> files.
> > 
> > 
> > 
> > On ffserver.conf-3cam-adsl-flv:
> > 
> > >This file should work as-is, except that the 
> > 
> > Missing # at beginning of line
> > 
> > ># cif size 
> > 
> > CIF

Done.

> > 
> > 
> > 
> > On ffserver.conf-swf:
> > 
> > ># Uses Flash Player, works wtih IE, and Firefox.
> > 
> > with. Or maybe the whole line could be removed, as
> > same
> > thing is said in header.

Done.

> > 
> > ># but Image size and quality 
> > ># Reduced for bandwidth. The client side 
> > 
> > # but image size and quality 
> > # reduced for bandwidth. The client side 
> > 

Done.

> > 
> > 
> > 
> > On ffserver.conf-swf-flv
> > 
> > >VideoBufferSize 128      # adsjust with bitrate,
> > and q.
> > 
> > Adjust
> > 

Done.

> > >VideoSize 352x288        # cif size
> > 
> > CIF

Done.




       
____________________________________________________________________________________
Choose the right car based on your needs.  Check out Yahoo! Autos new Car Finder tool.
http://autos.yahoo.com/carfinder/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ffserver-conf-files.tar
Type: application/x-tar
Size: 30720 bytes
Desc: 3205149320-ffserver-conf-files.tar
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070701/d0861ae3/attachment.tar>