Re: [FFmpeg-devel] [PATCH v8 1/1] avformat: Add IPFS protocol support.

[Michael Niedermayer <michael@niedermayer.cc>]

[-- Attachment #1.1: Type: text/plain, Size: 2614 bytes --]

On Tue, Mar 08, 2022 at 01:49:22PM +0100, Mark Gaiser wrote:
> On Fri, Mar 4, 2022 at 7:09 PM Michael Niedermayer <michael@niedermayer.cc>
> wrote:
> 
> > On Thu, Mar 03, 2022 at 03:58:53PM +0100, Mark Gaiser wrote:
> > > On Tue, Mar 1, 2022 at 11:01 PM Michael Niedermayer <
> > michael@niedermayer.cc>
> > > wrote:
> > >
> > > > On Mon, Feb 28, 2022 at 02:09:15PM +0100, Tomas Härdin wrote:
> > > > > sön 2022-02-27 klockan 15:29 +0100 skrev Mark Gaiser:
> > > > > > Ping 2....
> > > > > >
> > > > > > I'd really like to get this merged!
> > > > > > This kinda blocks me right now from proceeding with IPFS
> > integration
> > > > > > in
> > > > > > Kodi, MPV and VLC. Implementations in those (who rely on ffmpeg)
> > are
> > > > > > significantly easier once this patch is finally landed in ffmpeg.
> > > > >
> > > > > I'd like to hear at least one other dev chime in on this one
> > > >
> > > > what exactly are you not sure about ?
> > > > what exactly needs a 2nd look ?
> > > >
> > >
> > > My assumption.
> > > In general just a second look by someone other than Tomas.
> > > And, as he was skeptical about this patch at first, likely another
> > opinion
> > > if this makes sense to add in ffmpeg.
> > > To me it does very much but i'm biased :)
> >
> > ipfs support makes sense to be added to ffmpeg. ive seen ipfs urls and ive
> > already been annoyed that some tools dont "just" work with them.
> > While if i compare this to many other formats which i have never seen
> > outside the context of FFmpeg. So from this biased single sample that i
> > am, ipfs seems more widespread and thats why iam in favor of its support
> >
> > thx
> >
> > Great to have your support :)
> Reading that is quite motivating to work on it, no joke!
> 
> Just to be clear here. Having this in ffmpeg won't make it "just work" yet.
> For a minimal feeling of "hey, it works out of the box" you'd need:
> - The next or version after the next IPFS.
> - MPV support which relies on this patch to even be supported in mpv
> - Have a node running locally

if theres no local node it should fallback to a public node
ATM
IPFS_GATEWAY=https://dweb.link ./ffplay ipfs://...
works
so such a fallback is all thats needed for it to just work

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

If the United States is serious about tackling the national security threats 
related to an insecure 5G network, it needs to rethink the extent to which it
values corporate profits and government espionage over security.-Bruce Schneier

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 bytes --]

[-- Attachment #2: Type: text/plain, Size: 251 bytes --]

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".