On Aug 12, 2014, at 8:37 PM, Simon Thelen <ffmpeg-...@c-14.de> wrote:
> On 12/08/14 at 10:36, Steven Liu wrote: >> 2014-08-12 7:26 GMT+08:00 Simon Thelen <ffmpeg-...@c-14.de>: >>> On 11/08/14 at 17:39, Steven Liu wrote: > [..] >>>> The FFmpeg hls module can make m3u8 and ts, but it dosen't delete the >>>> old ts segment file. >>>> If always run this module, the disk will full, so this patch can fix >>>> the problem. >>>> When update the segment list m3u8 file, it will delete the ts segment >>>> out range from the list file. >>> [..] >>> I'm pretty sure this is why the hls muxer has the `-hls_wrap wrap' option. > [..] >> hls_list_size and hls_wrap option two can fix the problem, but this >> patch can fix the problem of the default parameter, because the ts file on >> disk , one to one correspondence with the m3u8 file will better, isn't it? > IMO, the problem is that you're modifying the default behavior even > though an option already exists to let you specify the amount of segment > files to keep around. This might be suitable in your use-case, but what > if I (or someone else) wanted to keep all the segment files for archival > purposes, but not list them in the m3u8; with your patch this no longer > becomes possible. > > If you were to write a new patch that made hls_wrap default to > hls_list_size instead of the default 0, it would no longer impose > artificial limits on users. At that point, the inclusion of the patch > would depend on the opinion of the hls maintainer and the other FFmpeg > maintainers with opinions on the matter. Hi Simon Thelen, Maybe you’re right. I think the '-f segment' can save all of the segment ts file, so hls is used for living stream, so it not save segment ts be better; Maybe i’m wrong. but this patch just used for remove the segment ts not list in m3u8. :D > > -- > Simon Thelen > _______________________________________________ > 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