Re: [FFmpeg-devel] [PATCH] libavformat/segment: strftime date sub-directories

2021-12-21 Thread James Dutton
Hi, This is a follow on from: https://patchwork.ffmpeg.org/project/ffmpeg/patch/caamvbhg9qsksxfuzrt3resrr89j5scyd84ec6olearqx4cu...@mail.gmail.com/ On Tue, 21 Dec 2021 at 21:39, James Dutton wrote: > > Automatically create sub-directories if needed based on date. > E.g. > ffmpeg ... -timelimit

Re: [FFmpeg-devel] [PATCH] libavformat/segment: strftime date sub-directories

2018-09-27 Thread Steven Liu
Steven Liu 于2018年9月27日周四 下午2:53写道: > > Steven Liu 于2018年9月27日周四 上午6:37写道: > > > > > > > > > On Sep 27, 2018, at 06:10, Michael Niedermayer > > > wrote: > > > > > > On Wed, Sep 26, 2018 at 10:07:58AM +0100, James Courtier-Dutton wrote: > > >> On 26 September 2018 at 00:18, Steven Liu wrote: >

Re: [FFmpeg-devel] [PATCH] libavformat/segment: strftime date sub-directories

2018-09-27 Thread Steven Liu
Steven Liu 于2018年9月27日周四 上午6:37写道: > > > > > On Sep 27, 2018, at 06:10, Michael Niedermayer > > wrote: > > > > On Wed, Sep 26, 2018 at 10:07:58AM +0100, James Courtier-Dutton wrote: > >> On 26 September 2018 at 00:18, Steven Liu wrote: > >> > >>> > >>> > On Sep 26, 2018, at 06:49, James

Re: [FFmpeg-devel] [PATCH] libavformat/segment: strftime date sub-directories

2018-09-26 Thread Steven Liu
> On Sep 27, 2018, at 06:10, Michael Niedermayer wrote: > > On Wed, Sep 26, 2018 at 10:07:58AM +0100, James Courtier-Dutton wrote: >> On 26 September 2018 at 00:18, Steven Liu wrote: >> >>> >>> On Sep 26, 2018, at 06:49, James Courtier-Dutton >>> wrote: On 25 September 2018 at

Re: [FFmpeg-devel] [PATCH] libavformat/segment: strftime date sub-directories

2018-09-26 Thread Michael Niedermayer
On Wed, Sep 26, 2018 at 10:07:58AM +0100, James Courtier-Dutton wrote: > On 26 September 2018 at 00:18, Steven Liu wrote: > > > > > > > > On Sep 26, 2018, at 06:49, James Courtier-Dutton > > wrote: > > > On 25 September 2018 at 23:24, Steven Liu wrote: > > yes, the error message is confusing,

Re: [FFmpeg-devel] [PATCH] libavformat/segment: strftime date sub-directories

2018-09-26 Thread Steven Liu
James Courtier-Dutton 于2018年9月26日周三 下午5:08写道: > > On 26 September 2018 at 00:18, Steven Liu wrote: > > > > > > > > On Sep 26, 2018, at 06:49, James Courtier-Dutton > > wrote: > > > On 25 September 2018 at 23:24, Steven Liu wrote: > > yes, the error message is confusing, maybe change it here is

Re: [FFmpeg-devel] [PATCH] libavformat/segment: strftime date sub-directories

2018-09-26 Thread James Courtier-Dutton
On 26 September 2018 at 00:18, Steven Liu wrote: > > > > On Sep 26, 2018, at 06:49, James Courtier-Dutton > wrote: > > On 25 September 2018 at 23:24, Steven Liu wrote: > yes, the error message is confusing, maybe change it here is better. > > > > +av_log(oc, AV_LOG_ERROR, "Could

Re: [FFmpeg-devel] [PATCH] libavformat/segment: strftime date sub-directories

2018-09-25 Thread Steven Liu
> On Sep 26, 2018, at 06:49, James Courtier-Dutton > wrote: > > > > On 25 September 2018 at 23:24, Steven Liu wrote: > @@ -200,12 +200,27 @@ static int set_segment_filename(AVFormatContext *s) > if (seg->use_strftime) { > time_t now0; > struct tm *tm, tmpbuf; > +

Re: [FFmpeg-devel] [PATCH] libavformat/segment: strftime date sub-directories

2018-09-25 Thread Steven Liu
> On Sep 26, 2018, at 00:52, James Courtier-Dutton > wrote: > >> >> Automatically create sub-directories if needed based on date. >> E.g. >> ffmpeg ... -timelimit 2147483647 -f segment -strftime 1 -segment_time 10 >> "%Y/%m/%d/%Y-%m-%d_%H-%M-%S.mkv" >> >> >> >> >> Maybe you want add an

Re: [FFmpeg-devel] [PATCH] libavformat/segment: strftime date sub-directories

2018-09-25 Thread James Courtier-Dutton
> > Automatically create sub-directories if needed based on date. > E.g. > ffmpeg ... -timelimit 2147483647 -f segment -strftime 1 -segment_time 10 > "%Y/%m/%d/%Y-%m-%d_%H-%M-%S.mkv" > > > > > Maybe you want add an option named “strftime_mkdir” not > “use_localtime_mkdir”. > You should add the

Re: [FFmpeg-devel] [PATCH] libavformat/segment: strftime date sub-directories

2018-09-25 Thread Steven Liu
> On Sep 25, 2018, at 22:37, James Courtier-Dutton > wrote: > > Updated patch. > This one applies on top of > commit bb660800a55f5171f77316941afe2e62534ee72c > > > On 25 September 2018 at 15:00, Steven Liu wrote: > >> >> >>> On Sep 25, 2018, at 20:33, James Courtier-Dutton >> wrote:

Re: [FFmpeg-devel] [PATCH] libavformat/segment: strftime date sub-directories

2018-09-25 Thread James Courtier-Dutton
Updated patch. This one applies on top of commit bb660800a55f5171f77316941afe2e62534ee72c On 25 September 2018 at 15:00, Steven Liu wrote: > > > > On Sep 25, 2018, at 20:33, James Courtier-Dutton > wrote: > > > > On 25 September 2018 at 10:22, Moritz Barsnick wrote: > > > >> > >> Steven

Re: [FFmpeg-devel] [PATCH] libavformat/segment: strftime date sub-directories

2018-09-25 Thread Steven Liu
> On Sep 25, 2018, at 20:33, James Courtier-Dutton > wrote: > > On 25 September 2018 at 10:22, Moritz Barsnick wrote: > >> >> Steven already posted a patch proposal to this list, factoring out the >> function: >> https://patchwork.ffmpeg.org/patch/10471/ >>

Re: [FFmpeg-devel] [PATCH] libavformat/segment: strftime date sub-directories

2018-09-25 Thread James Courtier-Dutton
On 25 September 2018 at 10:22, Moritz Barsnick wrote: > > Steven already posted a patch proposal to this list, factoring out the > function: > https://patchwork.ffmpeg.org/patch/10471/ > http://ffmpeg.org/pipermail/ffmpeg-devel/2018-September/234538.html > > That looks fine by me. I can resubmit

Re: [FFmpeg-devel] [PATCH] libavformat/segment: strftime date sub-directories

2018-09-25 Thread Moritz Barsnick
On Tue, Sep 25, 2018 at 08:49:25 +, Jeyapal, Karthick wrote: > > It is a duplicate from hisenc.c. hisenc already has this "create > > sub-directories" feature, but I needed it for the segment use case. > > I made it static so that it would be treated as local to the .c file. > > I don't know

Re: [FFmpeg-devel] [PATCH] libavformat/segment: strftime date sub-directories

2018-09-25 Thread Jeyapal, Karthick
On 9/25/18 12:28 AM, James Courtier-Dutton wrote: > On 24 September 2018 at 13:48, Moritz Barsnick wrote: > >> On Mon, Sep 24, 2018 at 12:39:42 +0100, James Courtier-Dutton wrote: >> >>> Automatically create sub-directories if needed based on date. >>> E.g. >>> ffmpeg ... -timelimit 2147483647

Re: [FFmpeg-devel] [PATCH] libavformat/segment: strftime date sub-directories

2018-09-24 Thread James Courtier-Dutton
On 24 September 2018 at 13:48, Moritz Barsnick wrote: > On Mon, Sep 24, 2018 at 12:39:42 +0100, James Courtier-Dutton wrote: > > > Automatically create sub-directories if needed based on date. > > E.g. > > ffmpeg ... -timelimit 2147483647 -f segment -strftime 1 -segment_time 10 >

Re: [FFmpeg-devel] [PATCH] libavformat/segment: strftime date sub-directories

2018-09-24 Thread Moritz Barsnick
On Mon, Sep 24, 2018 at 12:39:42 +0100, James Courtier-Dutton wrote: > Automatically create sub-directories if needed based on date. > E.g. > ffmpeg ... -timelimit 2147483647 -f segment -strftime 1 -segment_time 10 > "%Y/%m/%d/%Y-%m-%d_%H-%M-%S.mkv" [...] > +static int mkdir_p(const char *path)

Re: [FFmpeg-devel] [PATCH] libavformat/segment: strftime date sub-directories

2018-09-24 Thread James Courtier-Dutton
Updated patch is here: https://patchwork.ffmpeg.org/patch/10469/ Functionalist has not changed. It has been updated to apply to the latest master branch. On 22 September 2018 at 17:26, Michael Niedermayer wrote: > On Thu, Nov 30, 2017 at 11:07:03PM +, James Courtier-Dutton wrote: > > > >

Re: [FFmpeg-devel] [PATCH] libavformat/segment: strftime date sub-directories

2018-09-24 Thread James Courtier-Dutton
From 6c74670e047bfc9e995c0f968c1688be768d13bd Mon Sep 17 00:00:00 2001 From: James Courtier-Dutton Date: Mon, 24 Sep 2018 12:34:54 +0100 Subject: [PATCH] avformat/segment: strftime date sub-directories Automatically create sub-directories if needed based on date. E.g. ffmpeg ... -timelimit

Re: [FFmpeg-devel] [PATCH] libavformat/segment: strftime date sub-directories

2018-09-22 Thread Michael Niedermayer
On Thu, Nov 30, 2017 at 11:07:03PM +, James Courtier-Dutton wrote: > > segment.c | 48 > 1 file changed, 48 insertions(+) > e3e9961df186f73adb24f7a334d398884e894d53 > 0001-libavformat-segment-strftime-date-sub-directories.patch > From