diff mbox

[FFmpeg-devel] doc/hlsenc: hls_segment_filename, use_localtime, , use_localtime_mkdir, hls_flags

Message ID 9654fd55-5d41-8254-e5fc-627dd8cde28c@vivanet.hu
State Superseded
Headers show

Commit Message

Bodecs Bela Jan. 2, 2017, 11:49 a.m. UTC
2017.01.02. 1:29 keltezéssel, Steven Liu írta:
> 2017-01-02 7:22 GMT+08:00 Bodecs Bela <bodecsb@vivanet.hu>:
>
>>
>> 2017.01.02. 0:12 keltezéssel, Steven Liu írta:
>>
>>> 2017-01-02 5:53 GMT+08:00 Bodecs Bela <bodecsb@vivanet.hu>:
>>>
>>> Dear All,
>>>> I added some more details and example to hls muxer documentation. I
>>>> reformatted hls_flags option list to table.
>>>>
>>>> Where is the patch?
>> I am sorry. I have atteched it now,
>>
>>
>>>> thank you,
>>>>
>>>> Bela Bodecs
>>>>
>>>>
>>>> _______________________________________________
>>>> ffmpeg-devel mailing list
>>>> ffmpeg-devel@ffmpeg.org
>>>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>>>
>>>> _______________________________________________
>>> ffmpeg-devel mailing list
>>> ffmpeg-devel@ffmpeg.org
>>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>>
>>
>> _______________________________________________
>> ffmpeg-devel mailing list
>> ffmpeg-devel@ffmpeg.org
>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>
>>
> title:  doc/hlsenc  to doc/muxers
>
> text:
> -Used together with -use_localtime, it will create up to one subdirectory
> which
> +Used together with use_localtime, it will create all subdirectories which
> change to
> -Used together with -use_localtime, it will create up to one subdirectory
> which
> +Used together with -use_localtime, it will create all subdirectories which
I have corrected.
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Comments

Steven Liu Jan. 2, 2017, 11:52 a.m. UTC | #1
2017-01-02 19:49 GMT+08:00 Bodecs Bela <bodecsb@vivanet.hu>:

>
>
> 2017.01.02. 1:29 keltezéssel, Steven Liu írta:
>
>> 2017-01-02 7:22 GMT+08:00 Bodecs Bela <bodecsb@vivanet.hu>:
>>
>>
>>> 2017.01.02. 0:12 keltezéssel, Steven Liu írta:
>>>
>>> 2017-01-02 5:53 GMT+08:00 Bodecs Bela <bodecsb@vivanet.hu>:
>>>>
>>>> Dear All,
>>>>
>>>>> I added some more details and example to hls muxer documentation. I
>>>>> reformatted hls_flags option list to table.
>>>>>
>>>>> Where is the patch?
>>>>>
>>>> I am sorry. I have atteched it now,
>>>
>>>
>>> thank you,
>>>>>
>>>>> Bela Bodecs
>>>>>
>>>>>
>>>>> _______________________________________________
>>>>> ffmpeg-devel mailing list
>>>>> ffmpeg-devel@ffmpeg.org
>>>>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>>>>
>>>>> _______________________________________________
>>>>>
>>>> ffmpeg-devel mailing list
>>>> ffmpeg-devel@ffmpeg.org
>>>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>>>
>>>>
>>> _______________________________________________
>>> ffmpeg-devel mailing list
>>> ffmpeg-devel@ffmpeg.org
>>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>>
>>>
>>> title:  doc/hlsenc  to doc/muxers
>>
>> text:
>> -Used together with -use_localtime, it will create up to one subdirectory
>> which
>> +Used together with use_localtime, it will create all subdirectories which
>> change to
>> -Used together with -use_localtime, it will create up to one subdirectory
>> which
>> +Used together with -use_localtime, it will create all subdirectories
>> which
>>
> I have corrected.
>
> _______________________________________________
>> ffmpeg-devel mailing list
>> ffmpeg-devel@ffmpeg.org
>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>
>
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
>
LGTM
Moritz Barsnick Jan. 2, 2017, 12:54 p.m. UTC | #2
On Mon, Jan 02, 2017 at 12:49:50 +0100, Bodecs Bela wrote:
> Subject: [FFmpeg-devel] [PATCH] doc/hlsenc: hls_segment_filename, use_localtime, , use_localtime_mkdir, hls_flags
                                                                                   ^^
There's a ", " too much in that part of the commit message.

> +@var{filename} may contain full path or relative path specification, but only the file name part without any path info will be contained in the m3u8 segment list.

You should break this line.

> +Should relative path is specified, path of created segment files will be relative to the current working directory.

-> 
  Should a relative path be specified, the path of the created segment
  files will be relative to the current working directory.

> +When -use_localtime_mkdir is set, the whole expanded value of @var{filename} will be written into m3u8 segment list.
        ^ I think you should omit the dashes, as these are primarily
          library options, not command line options.

And do add an article: "...  will be written into the m3u8 segment
list."

> -@file{file-20160215-1455569023.ts}, @file{file-20160215-1455569024.ts}, etc.
> +@file{file-20160215-1455569023.ts}, @file{file-20160215-1455569024.ts}, etc. (Notice: in some systems/environments %s specifier is not available. See strftime() documentation!)

->
  Note: On some systems/environments, the code{%s} specifier is not available. See
  code{strftime()} documentation.

(Avoid exclamation marks.)

> +This example will create a directory hierarchy 2016/02/15 (if any of them does not exist), and then
                                                                             ^ do

Moritz
diff mbox

Patch

From 18a85221d6c7044baa0e40250bf18dfe165da777 Mon Sep 17 00:00:00 2001
From: Bela Bodecs <bodecsb@vivanet.hu>
Date: Sun, 1 Jan 2017 22:48:27 +0100
Subject: [PATCH] doc/muxers: hls_segment_filename, use_localtime,
 use_localtime_mkdir

added some more details and example. hls_flags option list reformatted
to table

Signed-off-by: Bela Bodecs <bodecsb@vivanet.hu>
---
 doc/muxers.texi | 45 ++++++++++++++++++++++++++++++++-------------
 1 file changed, 32 insertions(+), 13 deletions(-)

diff --git a/doc/muxers.texi b/doc/muxers.texi
index c2598b2..3cb2918 100644
--- a/doc/muxers.texi
+++ b/doc/muxers.texi
@@ -441,15 +441,20 @@  ffmpeg -i in.nut -hls_segment_filename 'file%03d.ts' out.m3u8
 This example will produce the playlist, @file{out.m3u8}, and segment files:
 @file{file000.ts}, @file{file001.ts}, @file{file002.ts}, etc.
 
+@var{filename} may contain full path or relative path specification, but only the file name part without any path info will be contained in the m3u8 segment list.
+Should relative path is specified, path of created segment files will be relative to the current working directory.
+When -use_localtime_mkdir is set, the whole expanded value of @var{filename} will be written into m3u8 segment list.
+
+
 @item use_localtime
-Use strftime on @var{filename} to expand the segment filename with localtime.
+Use strftime() on @var{filename} to expand the segment filename with localtime.
 The segment number is also available in this mode, but to use it, you need to specify second_level_segment_index
 hls_flag and %%d will be the specifier.
 @example
 ffmpeg -i in.nut -use_localtime 1 -hls_segment_filename 'file-%Y%m%d-%s.ts' out.m3u8
 @end example
 This example will produce the playlist, @file{out.m3u8}, and segment files:
-@file{file-20160215-1455569023.ts}, @file{file-20160215-1455569024.ts}, etc.
+@file{file-20160215-1455569023.ts}, @file{file-20160215-1455569024.ts}, etc. (Notice: in some systems/environments %s specifier is not available. See strftime() documentation!)
 @example
 ffmpeg -i in.nut -use_localtime 1 -hls_flags second_level_segment_index -hls_segment_filename 'file-%Y%m%d-%%04d.ts' out.m3u8
 @end example
@@ -457,14 +462,21 @@  This example will produce the playlist, @file{out.m3u8}, and segment files:
 @file{file-20160215-0001.ts}, @file{file-20160215-0002.ts}, etc.
 
 @item use_localtime_mkdir
-Used together with -use_localtime, it will create up to one subdirectory which
+Used together with -use_localtime, it will create all subdirectories which
 is expanded in @var{filename}.
 @example
 ffmpeg -i in.nut -use_localtime 1 -use_localtime_mkdir 1 -hls_segment_filename '%Y%m%d/file-%Y%m%d-%s.ts' out.m3u8
 @end example
 This example will create a directory 201560215 (if it does not exist), and then
 produce the playlist, @file{out.m3u8}, and segment files:
-@file{201560215/file-20160215-1455569023.ts}, @file{201560215/file-20160215-1455569024.ts}, etc.
+@file{20160215/file-20160215-1455569023.ts}, @file{20160215/file-20160215-1455569024.ts}, etc.
+
+@example
+ffmpeg -i in.nut -use_localtime 1 -use_localtime_mkdir 1 -hls_segment_filename '%Y/%m/%d/file-%Y%m%d-%s.ts' out.m3u8
+@end example
+This example will create a directory hierarchy 2016/02/15 (if any of them does not exist), and then
+produce the playlist, @file{out.m3u8}, and segment files:
+@file{2016/02/15/file-20160215-1455569023.ts}, @file{2016/02/15/file-20160215-1455569024.ts}, etc.
 
 
 @item hls_key_info_file @var{key_info_file}
@@ -523,7 +535,12 @@  ffmpeg -f lavfi -re -i testsrc -c:v h264 -hls_flags delete_segments \
   -hls_key_info_file file.keyinfo out.m3u8
 @end example
 
-@item hls_flags single_file
+
+@item hls_flags @var{flags}
+Possible values:
+
+@table @samp
+@item single_file
 If this flag is set, the muxer will store all segments in a single MPEG-TS
 file, and will use byte ranges in the playlist. HLS playlists generated with
 this way will have the version number 4.
@@ -534,37 +551,39 @@  ffmpeg -i in.nut -hls_flags single_file out.m3u8
 Will produce the playlist, @file{out.m3u8}, and a single segment file,
 @file{out.ts}.
 
-@item hls_flags delete_segments
+@item delete_segments
 Segment files removed from the playlist are deleted after a period of time
 equal to the duration of the segment plus the duration of the playlist.
 
-@item hls_flags append_list
+@item append_list
 Append new segments into the end of old segment list,
 and remove the @code{#EXT-X-ENDLIST} from the old segment list.
 
-@item hls_flags round_durations
+@item round_durations
 Round the duration info in the playlist file segment info to integer
 values, instead of using floating point.
 
-@item hls_flags discont_starts
+@item discont_starts
 Add the @code{#EXT-X-DISCONTINUITY} tag to the playlist, before the
 first segment's information.
 
-@item hls_flags omit_endlist
+@item omit_endlist
 Do not append the @code{EXT-X-ENDLIST} tag at the end of the playlist.
 
-@item hls_flags split_by_time
+@item split_by_time
 Allow segments to start on frames other than keyframes. This improves
 behavior on some players when the time between keyframes is inconsistent,
 but may make things worse on others, and can cause some oddities during
 seeking. This flag should be used with the @code{hls_time} option.
 
-@item hls_flags program_date_time
+@item program_date_time
 Generate @code{EXT-X-PROGRAM-DATE-TIME} tags.
 
-@item hls_flags second_level_segment_index
+@item second_level_segment_index
 Makes it possible to use segment indexes as %%d besides date/time values when use_localtime is on.
 
+@end table
+
 @item hls_playlist_type event
 Emit @code{#EXT-X-PLAYLIST-TYPE:EVENT} in the m3u8 header. Forces
 @option{hls_list_size} to 0; the playlist can only be appended to.
-- 
2.5.3.windows.1