[FFmpeg-devel] [FFmpeg-cvslog] avformat/hlsenc: replace with av_dirname to get the directory

Limin Wang lance.lmwang at gmail.com
Tue Oct 8 17:35:29 EEST 2019


On Tue, Oct 08, 2019 at 02:47:57PM +0200, Michael Niedermayer wrote:
> On Tue, Oct 08, 2019 at 06:24:22AM +0000, Limin Wang wrote:
> > ffmpeg | branch: master | Limin Wang <lance.lmwang at gmail.com> | Tue Sep 24 19:24:00 2019 +0800| [28bb73cee77e209a6445d081e7ea577ede29c50f] | committer: Steven Liu
> > 
> > avformat/hlsenc: replace with av_dirname to get the directory
> > 
> > Signed-off-by: Limin Wang <lance.lmwang at gmail.com>
> > 
> > > http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=28bb73cee77e209a6445d081e7ea577ede29c50f
> > ---
> > 
> >  libavformat/hlsenc.c | 22 ++++++----------------
> >  1 file changed, 6 insertions(+), 16 deletions(-)
> 
> This breaks
> ./ffmpeg -i mm-small.mpg -f hls -hls_list_size 5 -hls_flags +delete_segments  -acodec mp2 -vcodec mpeg2video  -ab 128k -hls_base_url delseg  /tmp/file-hlsdelseg.m3u8
> ...
> [hls muxer @ 0x2be16c0] failed to delete old segment /tmpfile-hlsdelseg0.ts: No such file or directory
> 
> using /tmp//file-hlsdelseg.m3u8 makes it work but that doesnt feel correct

Michael, I have submit a patch to fix the issue, please check it. The
old dirname have included with extra "/" in the string end, sorry I haven't
notice that. I have tested with your command.

> 
> thx
> 
> [...]
> -- 
> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
> 
> "I am not trying to be anyone's saviour, I'm trying to think about the
>  future and not be sad" - Elon Musk
> 



> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".



More information about the ffmpeg-devel mailing list