[FFmpeg-devel] [PATCH v6 3/3] avformat/hlsenc:addition of CODECS attribute in the master playlist

Dixit, Vishwanath vdixit at akamai.com
Mon Dec 18 15:12:06 EET 2017



>On 12/15/17, 8:19 PM, "Steven Liu" <lingjiujianke at gmail.com> wrote:
>
>2017-12-15 16:21 GMT+08:00 Jeyapal, Karthick <kjeyapal at akamai.com>:
>>
>>
>>>On 12/15/17, 4:45 AM, "Liu Steven" <lq at chinaffmpeg.org> wrote:
>>>
>>>
>>>> 在 2017年12月15日,上午6:28,Liu Steven <lq at chinaffmpeg.org> 写道:
>>>>
>>>>>
>>>>> 在 2017年12月15日,上午12:29,Jeyapal, Karthick <kjeyapal at akamai.com> 写道:
>>>>>
>>>>>> On 12/14/17, 8:24 PM, "Steven Liu" <lq at chinaffmpeg.org> wrote:
>>>>>>
>>>>>>
>>>>>>> 在 2017年12月14日,下午6:55,vdixit at akamai.com 写道:
>>>>>>> […]
>>>>>>> +libavformat/reverse.c
>>>>>>
>>>>>> this need double check for a better way
>>>>> The better way of doing this is to share ff_reverse function.
>>>> No I don’t think that is a better way here.
>>>>> But such a patch submitted recently was not pushed due to several objections. http://ffmpeg.org/pipermail/ffmpeg-devel/2017-December/221472.html
>>>>> Hence, we are left with only this option. Moreover like same approach was used for avcodec and avdevice(to add reverse.c), as well. So, extending the same approach for avformat shouldn’t deviate from ffmpeg’s principles.
>>>> I think there have better way, Please don’t define the ffmpeg’s principles to every place. you are duplicate the code from libavcodec//reverse.c to libavformat, IMHO, that is not a good option.
>>>I need some time to think about that.
>> Oh sure. Thanks.
>> In that case, could you please merge PATCH v6 1/3 alone, since that is a relatively independent feature.
>
>
>patch -p1 < ~/Downloads/FFmpeg-devel-v6-1-3-avformat-hlsenc-addition-of-EXT-X-MEDIA-tag-and-AUDIO-attribute.patch
>cd xxx/
>rm -rf *
>../configure --disable-network --disable-everything --enable-muxer=hls
>make
>
>
>CC libavformat/hlsenc.o
>src/libavformat/hlsenc.c:1142:21: error: use of undeclared identifier
>'master_pb'
>        avio_printf(master_pb, "#EXT-X-MEDIA:TYPE=AUDIO,GROUP-ID=\"group_%s\"",
>                    ^
>src/libavformat/hlsenc.c:1144:21: error: use of undeclared identifier
>'master_pb'
>        avio_printf(master_pb, ",NAME=\"audio_0\",DEFAULT=YES,URI=\"%s\"\n",
>                    ^
>2 errors generated.
>make: *** [libavformat/hlsenc.o] Error 1
>make: *** Waiting for unfinished jobs....
The patch needed rebasing since an hlsenc patch ([PATCH 3/3] avformat/hlsenc: Extend persistent http connections to playlists) was pushed on 15-dec. I have rebased the patch set and submitted with version v7. Could you please merge PATCH v7 1/3 (https://patchwork.ffmpeg.org/patch/6857/)



More information about the ffmpeg-devel mailing list