Re: [FFmpeg-devel] [PATCH] Adding mkdir option for img2enc (4th attempt)

2018-02-04 Thread Michael Niedermayer
On Sun, Feb 04, 2018 at 12:51:59PM +, Dr. Alan Barclay wrote: > On 17/01/2018 11:15, Carl Eugen Hoyos wrote: > >2018-01-17 11:56 GMT+01:00 Dr. Alan Barclay : > > > >>Attached in a further patch - adding the error checks. > > > >Please merge this patch into your previous

Re: [FFmpeg-devel] [PATCH] Adding mkdir option for img2enc (4th attempt)

2018-02-04 Thread Dr. Alan Barclay
On 17/01/2018 11:15, Carl Eugen Hoyos wrote: 2018-01-17 11:56 GMT+01:00 Dr. Alan Barclay : Attached in a further patch - adding the error checks. Please merge this patch into your previous patch. Both patches updated. Alan. And please avoid top-posting here, Carl

Re: [FFmpeg-devel] [PATCH] Adding mkdir option for img2enc (3rd attempt)

2018-01-17 Thread Marton Balint
On Wed, 17 Jan 2018, Dr. Alan Barclay wrote: Hi, Attached in a further patch - adding the error checks. Derek - I don't think there is a functional downside to this 'mkdir' option being a default behaviour, but it would introduce a minor performance penalty (which users maybe don't want).

Re: [FFmpeg-devel] [PATCH] Adding mkdir option for img2enc (3rd attempt)

2018-01-17 Thread Carl Eugen Hoyos
2018-01-17 11:56 GMT+01:00 Dr. Alan Barclay : > Attached in a further patch - adding the error checks. Please merge this patch into your previous patch. And please avoid top-posting here, Carl Eugen ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] [PATCH] Adding mkdir option for img2enc (3rd attempt)

2018-01-17 Thread Dr. Alan Barclay
Hi, Attached in a further patch - adding the error checks. Derek - I don't think there is a functional downside to this 'mkdir' option being a default behaviour, but it would introduce a minor performance penalty (which users maybe don't want). All comments and help appreciated. Thanks and

Re: [FFmpeg-devel] [PATCH] Adding mkdir option for img2enc (2nd attempt)

2017-12-27 Thread Derek Buitenhuis
Hi, On 12/27/2017 12:27 PM, Dr Alan Barclay wrote: > Resending the two (git format-patch) patches, without the top lines > removed (which I thought I needed to do as some patch emails didn't seem > to have them). > > All comments and help appreciated. [...] > Subject: [PATCH 1/2] Move

Re: [FFmpeg-devel] [PATCH] Adding mkdir option for img2enc (2nd attempt)

2017-12-27 Thread Dr Alan Barclay
Hi, Resending the two (git format-patch) patches, without the top lines removed (which I thought I needed to do as some patch emails didn't seem to have them). All comments and help appreciated. Thanks and Regards, Alan. On 27/12/17 01:41, Michael Niedermayer wrote: On Tue, Dec 26, 2017

Re: [FFmpeg-devel] [PATCH] Adding mkdir option for img2enc (2nd attempt)

2017-12-26 Thread Michael Niedermayer
On Tue, Dec 26, 2017 at 10:44:31PM +, Dr Alan Barclay wrote: > Hi All, > > Please would someone with an interest in img2enc take a look at my revised > patches for a minor feature addition and consider committing it to the main > line for me. > > Example: > ffmpeg -i ~/trailer.mp4 -strftime

[FFmpeg-devel] [PATCH] Adding mkdir option for img2enc (2nd attempt)

2017-12-26 Thread Dr Alan Barclay
Hi All, Please would someone with an interest in img2enc take a look at my revised patches for a minor feature addition and consider committing it to the main line for me. Example: ffmpeg -i ~/trailer.mp4 -strftime 1 -mkdir 1 %Y/%m/%d/out_%H-%M-%S.jpg Without the new mkdir option, this

Re: [FFmpeg-devel] [PATCH] Adding mkdir option for img2enc

2017-12-17 Thread Carl Eugen Hoyos
2017-12-17 23:41 GMT+01:00 Dr Alan Barclay : > Please would someone with an interest in img2enc take a look > at my minor feature addition and consider committing it to the > main line for me. To be acceptable, the patch has to be split in two and please move the definition

[FFmpeg-devel] [PATCH] Adding mkdir option for img2enc

2017-12-17 Thread Dr Alan Barclay
Hi All, Please would someone with an interest in img2enc take a look at my minor feature addition and consider committing it to the main line for me. Example: ffmpeg -i ~/trailer.mp4 -strftime 1 -mkdir 1 %Y/%m/%d/out_%H-%M-%S.jpg Without the new mkdir option, this command will fail if the