Re: [FFmpeg-devel] [PATCH] lavf: add zip protocol

2015-04-01 Thread Lukasz Marek
On 01.04.2015 21:22, Alexander Strasser wrote: On 2015-03-28 23:30 +0100, Lukasz Marek wrote: W dniu sobota, 28 marca 2015 Peter Ross napisał(a): What about the following? ffplay zip://dir/a.zip/m.mkv # open dir/a.zip, read file m.mkv inside ffplay -file m.mkv zip://dir/a.zip # s

Re: [FFmpeg-devel] [PATCH] lavf: add zip protocol

2015-04-01 Thread Alexander Strasser
On 2015-03-28 23:30 +0100, Lukasz Marek wrote: > W dniu sobota, 28 marca 2015 Peter Ross napisał(a): > > > On Sat, Mar 28, 2015 at 08:38:40PM +0100, Lukasz Marek wrote: > > > I assumed it is local file (no other option so far). So I stat full path > > > (/tmp/outer.zip/tmp/inner.zip/tmp/data.bin)

Re: [FFmpeg-devel] [PATCH] lavf: add zip protocol

2015-03-28 Thread Alexander Strasser
On 2015-03-29 00:32 +0100, Timo Rothenpieler wrote: > > Le nonidi 9 germinal, an CCXXIII, wm4 a écrit : > >> amovie1.zip, and then "/tmp/amovie1.zip//amovie.srt" is a perfectly > >> valid unix path. Multiple slashes are coalesced into 1. > > > > Just as you say it without realizing it, it is valid

Re: [FFmpeg-devel] [PATCH] lavf: add zip protocol

2015-03-28 Thread wm4
On Sun, 29 Mar 2015 00:20:53 +0100 Nicolas George wrote: > Le nonidi 9 germinal, an CCXXIII, wm4 a écrit : > > amovie1.zip, and then "/tmp/amovie1.zip//amovie.srt" is a perfectly > > valid unix path. Multiple slashes are coalesced into 1. > > Just as you say it without realizing it, it is valid

Re: [FFmpeg-devel] [PATCH] lavf: add zip protocol

2015-03-28 Thread Timo Rothenpieler
> Le nonidi 9 germinal, an CCXXIII, wm4 a écrit : >> amovie1.zip, and then "/tmp/amovie1.zip//amovie.srt" is a perfectly >> valid unix path. Multiple slashes are coalesced into 1. > > Just as you say it without realizing it, it is valid but useless. While it is useless, for example scripts freque

Re: [FFmpeg-devel] [PATCH] lavf: add zip protocol

2015-03-28 Thread Nicolas George
Le nonidi 9 germinal, an CCXXIII, wm4 a écrit : > amovie1.zip, and then "/tmp/amovie1.zip//amovie.srt" is a perfectly > valid unix path. Multiple slashes are coalesced into 1. Just as you say it without realizing it, it is valid but useless. Regards, -- Nicolas George signature.asc Descript

Re: [FFmpeg-devel] [PATCH] lavf: add zip protocol

2015-03-28 Thread wm4
On Sun, 29 Mar 2015 01:12:32 +0200 Ivan Kalvachev wrote: > On 3/28/15, Peter Ross wrote: > > On Sat, Mar 28, 2015 at 10:24:55PM +0100, wm4 wrote: > >> On Sun, 29 Mar 2015 08:10:29 +1100 > >> Peter Ross wrote: > >> > >> > On Sat, Mar 28, 2015 at 08:38:40PM +0100, Lukasz Marek wrote: > >> > > On

Re: [FFmpeg-devel] [PATCH] lavf: add zip protocol

2015-03-28 Thread Ivan Kalvachev
On 3/28/15, Peter Ross wrote: > On Sat, Mar 28, 2015 at 10:24:55PM +0100, wm4 wrote: >> On Sun, 29 Mar 2015 08:10:29 +1100 >> Peter Ross wrote: >> >> > On Sat, Mar 28, 2015 at 08:38:40PM +0100, Lukasz Marek wrote: >> > > On 28.03.2015 20:13, Nicolas George wrote: >> > > >L'octidi 8 germinal, an C

Re: [FFmpeg-devel] [PATCH] lavf: add zip protocol

2015-03-28 Thread Lukasz Marek
W dniu sobota, 28 marca 2015 Peter Ross napisał(a): > On Sat, Mar 28, 2015 at 08:38:40PM +0100, Lukasz Marek wrote: > > I assumed it is local file (no other option so far). So I stat full path > > (/tmp/outer.zip/tmp/inner.zip/tmp/data.bin) for being a file, if so then > I > > opened it as zip fi

Re: [FFmpeg-devel] [PATCH] lavf: add zip protocol

2015-03-28 Thread wm4
On Sun, 29 Mar 2015 08:55:42 +1100 Peter Ross wrote: > On Sat, Mar 28, 2015 at 10:24:55PM +0100, wm4 wrote: > > On Sun, 29 Mar 2015 08:10:29 +1100 > > Peter Ross wrote: > > > > > On Sat, Mar 28, 2015 at 08:38:40PM +0100, Lukasz Marek wrote: > > > > On 28.03.2015 20:13, Nicolas George wrote: > >

Re: [FFmpeg-devel] [PATCH] lavf: add zip protocol

2015-03-28 Thread Peter Ross
On Sat, Mar 28, 2015 at 10:24:55PM +0100, wm4 wrote: > On Sun, 29 Mar 2015 08:10:29 +1100 > Peter Ross wrote: > > > On Sat, Mar 28, 2015 at 08:38:40PM +0100, Lukasz Marek wrote: > > > On 28.03.2015 20:13, Nicolas George wrote: > > > >L'octidi 8 germinal, an CCXXIII, Lukasz Marek a écrit : > > > >

Re: [FFmpeg-devel] [PATCH] lavf: add zip protocol

2015-03-28 Thread wm4
On Sun, 29 Mar 2015 08:10:29 +1100 Peter Ross wrote: > On Sat, Mar 28, 2015 at 08:38:40PM +0100, Lukasz Marek wrote: > > On 28.03.2015 20:13, Nicolas George wrote: > > >L'octidi 8 germinal, an CCXXIII, Lukasz Marek a écrit : > > >>I will try to use this libarchive first and do some tests. Your ap

Re: [FFmpeg-devel] [PATCH] lavf: add zip protocol

2015-03-28 Thread Peter Ross
On Sat, Mar 28, 2015 at 08:38:40PM +0100, Lukasz Marek wrote: > On 28.03.2015 20:13, Nicolas George wrote: > >L'octidi 8 germinal, an CCXXIII, Lukasz Marek a écrit : > >>I will try to use this libarchive first and do some tests. Your approach may > >>collapse in case compression libraries doesn't s

Re: [FFmpeg-devel] [PATCH] lavf: add zip protocol

2015-03-28 Thread Lukasz Marek
On 28.03.2015 20:13, Nicolas George wrote: L'octidi 8 germinal, an CCXXIII, Lukasz Marek a écrit : I will try to use this libarchive first and do some tests. Your approach may collapse in case compression libraries doesn't support parallel compression/decompression (I mean that you write or read

Re: [FFmpeg-devel] [PATCH] lavf: add zip protocol

2015-03-28 Thread Nicolas George
L'octidi 8 germinal, an CCXXIII, Lukasz Marek a écrit : > I will try to use this libarchive first and do some tests. Your approach may > collapse in case compression libraries doesn't support parallel > compression/decompression (I mean that you write or read several files from > single archive fil

Re: [FFmpeg-devel] [PATCH] lavf: add zip protocol

2015-03-28 Thread Lukasz Marek
On 28.03.2015 11:53, Nicolas George wrote: Le septidi 7 germinal, an CCXXIII, Lukasz Marek a écrit : But, this time I dont understand you comments, could you elaborate it? What's wrong, what can I do? What I am saying is that there are a lot of different cases where we want to read archives (n

Re: [FFmpeg-devel] [PATCH] lavf: add zip protocol

2015-03-28 Thread Nicolas George
Le septidi 7 germinal, an CCXXIII, Lukasz Marek a écrit : > But, this time I dont understand you comments, could you elaborate it? > What's wrong, what can I do? What I am saying is that there are a lot of different cases where we want to read archives (not only zip, see my previous mail, but that

Re: [FFmpeg-devel] [PATCH] lavf: add zip protocol

2015-03-27 Thread Lukasz Marek
On 27.03.2015 15:49, Nicolas George wrote: Le septidi 7 germinal, an CCXXIII, Lukasz Marek a écrit : TODO: add doc, update doc/APIChanges, bump minor This is interesting, but the use case must be considered very carefully, there are tons of things that can be done with a zip file, even relevan

Re: [FFmpeg-devel] [PATCH] lavf: add zip protocol

2015-03-27 Thread Nicolas George
Le septidi 7 germinal, an CCXXIII, Lukasz Marek a écrit : > TODO: add doc, update doc/APIChanges, bump minor This is interesting, but the use case must be considered very carefully, there are tons of things that can be done with a zip file, even relevant to FFmpeg: - demux a bunch of images as a

[FFmpeg-devel] [PATCH] lavf: add zip protocol

2015-03-27 Thread Lukasz Marek
TODO: add doc, update doc/APIChanges, bump minor --- configure| 4 ++ libavformat/Makefile | 2 + libavformat/allformats.c | 1 + libavformat/zip.c| 176 +++ 4 files changed, 183 insertions(+) create mode 100644 libavfo