diff mbox

[FFmpeg-devel,2/2] add option hls_init_time document

Message ID CADxeRwmU1yPEZt78cWKeTmLESeX_NsJJhTTY7a1x8shMxf1Z-g@mail.gmail.com
State Accepted
Headers show

Commit Message

Steven Liu Aug. 26, 2016, 11:42 p.m. UTC
2016-08-27 6:00 GMT+08:00 Ibrahim Tachijian <barhom@netsat.se>:

> What do you think about something like this?
>
> diff --git a/doc/muxers.texi b/doc/muxers.texi
> index 2548aeb..c39a6a0 100644
> --- a/doc/muxers.texi
> +++ b/doc/muxers.texi
> @@ -357,6 +357,12 @@ segmentation.
>  This muxer supports the following options:
>
>  @table @option
> +@item hls_init_time @var{seconds}
> +Set the initial target segment length in seconds. Default value is
> @var{0}.
> +Segment will be cut on the next key frame after this time has passed on
> the first m3u8 list.
> +After the initial playlist is filled @command{ffmpeg} will cut segments
> +at duration equal to @code{hls_time}
> +
>  @item hls_time @var{seconds}
>  Set the target segment length in seconds. Default value is 2.
>  Segment will be cut on the next key frame after this time has passed.
>
> On Fri, Aug 26, 2016 at 11:32 AM Steven Liu <lingjiujianke@gmail.com>
> wrote:
>
>>
>> 2016-08-26 17:31 GMT+08:00 Steven Liu <lingjiujianke@gmail.com>:
>>
>>>
>>> Signed-off-by: LiuQi <liuqi@gosun.com>
>>> ---
>>>  doc/muxers.texi | 7 +++++++
>>>  1 file changed, 7 insertions(+)
>>>
>>> diff --git a/doc/muxers.texi b/doc/muxers.texi
>>> index 2548aeb..4db3809 100644
>>> --- a/doc/muxers.texi
>>> +++ b/doc/muxers.texi
>>> @@ -357,6 +357,13 @@ segmentation.
>>>  This muxer supports the following options:
>>>
>>>  @table @option
>>> +@item hls_init_time @var{seconds}
>>> +Set the target segment length in seconds. Default value is @var{0}.
>>> +Segment will be cut duration is @code{hls_init_time} value for the
>>> +first m3u8 list segments, After update m3u8 to the end of the
>>> +first m3u8 list,  @command{ffmpeg} will cut segments
>>> +at duration equ @code{hls_time}.
>>> +
>>>  @item hls_time @var{seconds}
>>>  Set the target segment length in seconds. Default value is 2.
>>>  Segment will be cut on the next key frame after this time has passed.
>>> --
>>> 2.7.4 (Apple Git-66)
>>>
>>> Hi Ibrahim Tachijian,
>>
>>     Can you review this doc? My English is poor :D
>>
> Reviewed-by: Ibrahim Tachijian <barhom@netsat.se>
Signed-off-by: LiuQi <liuqi@gosun.com>
---
 doc/muxers.texi | 6 ++++++
 1 file changed, 6 insertions(+)

 Segment will be cut on the next key frame after this time has passed.

Comments

Michael Niedermayer Aug. 29, 2016, 1:05 p.m. UTC | #1
On Sat, Aug 27, 2016 at 07:42:18AM +0800, Steven Liu wrote:
> 2016-08-27 6:00 GMT+08:00 Ibrahim Tachijian <barhom@netsat.se>:
> 
> > What do you think about something like this?
> >
> > diff --git a/doc/muxers.texi b/doc/muxers.texi
> > index 2548aeb..c39a6a0 100644
> > --- a/doc/muxers.texi
> > +++ b/doc/muxers.texi
> > @@ -357,6 +357,12 @@ segmentation.
> >  This muxer supports the following options:
> >
> >  @table @option
> > +@item hls_init_time @var{seconds}
> > +Set the initial target segment length in seconds. Default value is
> > @var{0}.
> > +Segment will be cut on the next key frame after this time has passed on
> > the first m3u8 list.
> > +After the initial playlist is filled @command{ffmpeg} will cut segments
> > +at duration equal to @code{hls_time}
> > +
> >  @item hls_time @var{seconds}
> >  Set the target segment length in seconds. Default value is 2.
> >  Segment will be cut on the next key frame after this time has passed.
> >
> > On Fri, Aug 26, 2016 at 11:32 AM Steven Liu <lingjiujianke@gmail.com>
> > wrote:
> >
> >>
> >> 2016-08-26 17:31 GMT+08:00 Steven Liu <lingjiujianke@gmail.com>:
> >>
> >>>
> >>> Signed-off-by: LiuQi <liuqi@gosun.com>
> >>> ---
> >>>  doc/muxers.texi | 7 +++++++
> >>>  1 file changed, 7 insertions(+)
> >>>
> >>> diff --git a/doc/muxers.texi b/doc/muxers.texi
> >>> index 2548aeb..4db3809 100644
> >>> --- a/doc/muxers.texi
> >>> +++ b/doc/muxers.texi
> >>> @@ -357,6 +357,13 @@ segmentation.
> >>>  This muxer supports the following options:
> >>>
> >>>  @table @option
> >>> +@item hls_init_time @var{seconds}
> >>> +Set the target segment length in seconds. Default value is @var{0}.
> >>> +Segment will be cut duration is @code{hls_init_time} value for the
> >>> +first m3u8 list segments, After update m3u8 to the end of the
> >>> +first m3u8 list,  @command{ffmpeg} will cut segments
> >>> +at duration equ @code{hls_time}.
> >>> +
> >>>  @item hls_time @var{seconds}
> >>>  Set the target segment length in seconds. Default value is 2.
> >>>  Segment will be cut on the next key frame after this time has passed.
> >>> --
> >>> 2.7.4 (Apple Git-66)
> >>>
> >>> Hi Ibrahim Tachijian,
> >>
> >>     Can you review this doc? My English is poor :D
> >>
> > Reviewed-by: Ibrahim Tachijian <barhom@netsat.se>
> Signed-off-by: LiuQi <liuqi@gosun.com>
> ---
>  doc/muxers.texi | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/doc/muxers.texi b/doc/muxers.texi
> index 2548aeb..c39a6a0 100644
> --- a/doc/muxers.texi
> +++ b/doc/muxers.texi
> @@ -357,6 +357,12 @@ segmentation.
>  This muxer supports the following options:
> 
>  @table @option
> +@item hls_init_time @var{seconds}
> +Set the initial target segment length in seconds. Default value is @var{0}.
> +Segment will be cut on the next key frame after this time has passed on
> the first m3u8 list.
> +After the initial playlist is filled @command{ffmpeg} will cut segments
> +at duration equal to @code{hls_time}
> +
>  @item hls_time @var{seconds}
>  Set the target segment length in seconds. Default value is 2.
>  Segment will be cut on the next key frame after this time has passed.
> -- 
> 2.7.4 (Apple Git-66)

>  muxers.texi |    6 ++++++
>  1 file changed, 6 insertions(+)
> 9c534308910fa4559ecd159dfc461e9b1085d3ab  0002-add-option-hls_init_time-document.patch
> From a67c3c43a2ee58156b5817d26632ec6c15756957 Mon Sep 17 00:00:00 2001
> From: Steven Liu <lingjiujianke@gmail.com>
> Date: Sat, 27 Aug 2016 07:35:54 +0800
> Subject: [PATCH 2/2] add option hls_init_time document
> 
> Reviewed-by: Ibrahim Tachijian <barhom@netsat.se>

applied

thx


[...]
diff mbox

Patch

diff --git a/doc/muxers.texi b/doc/muxers.texi
index 2548aeb..c39a6a0 100644
--- a/doc/muxers.texi
+++ b/doc/muxers.texi
@@ -357,6 +357,12 @@  segmentation.
 This muxer supports the following options:

 @table @option
+@item hls_init_time @var{seconds}
+Set the initial target segment length in seconds. Default value is @var{0}.
+Segment will be cut on the next key frame after this time has passed on
the first m3u8 list.
+After the initial playlist is filled @command{ffmpeg} will cut segments
+at duration equal to @code{hls_time}
+
 @item hls_time @var{seconds}
 Set the target segment length in seconds. Default value is 2.