From: Mark Gaiser <markg85@gmail.com> To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org> Subject: Re: [FFmpeg-devel] [PATCH 0/5] Add IPFS and IPNS protocol support Date: Mon, 31 Jan 2022 23:00:22 +0100 Message-ID: <CAPd6JnHcMZajgdh7Ayo5BwsW8G6N7namQj0RwvxLzGv1H4io8g@mail.gmail.com> (raw) In-Reply-To: <af3c20a01f52f6c0ef3fc90958cc655980533884.camel@acc.umu.se> On Mon, Jan 31, 2022 at 9:23 PM Tomas Härdin <tjoppen@acc.umu.se> wrote: > mån 2022-01-31 klockan 17:31 +0100 skrev Mark Gaiser: > > On Mon, Jan 31, 2022 at 4:52 PM Tomas Härdin <tjoppen@acc.umu.se> > > wrote: > > > > > mån 2022-01-31 klockan 14:51 +0100 skrev Mark Gaiser: > > > > > > > > There are multiple ways to access files on the IPFS network. This > > > > patch series > > > > uses the gateway driven way. An IPFS node - by default - exposes > > > > a > > > > local > > > > gateway (say http://localhost:8080) which is then used to get > > > > content > > > > from IPFS. > > > > > > > > > Perhaps the protocol should be called something other than just > > > ipfs if > > > it doesn't actually implement IPFS. Like ipfsgateway. It could > > > still be > > > registered to ipfs:// of course, until someone writes a wrapper for > > > libipfs. > > > > > > > Do you mean to have it named like "ipfsgateway" as files (and > > library) but > > keep the protocol registration of ipfs and ipns? > > I'm fine with that. The name is only artificial in code anyhow, all > > that > > matters are the protocol names. > > What I'm really after is if other devs think there might be an issue > once someone goes an implements proper IPFS support > A "proper" implementation is unfeasible for ffmpeg purposes because a proper implementation would act as an IPFS node. That means it would: - spin up - do it's bootstrapping - connect to nodes and find new nodes to connect to - find the CID on the network - etc... This all adds a lot of startup time making it very unfriendly to users. In this scenario it could take up to minutes before your video starts playing if it doesn't time out. Rather, the gateway approach uses an already running IPFS instance. Users are meant to run a local IPFS instance but it doesn't *have* to be local. Sure, a user could point it to another local network hosted IPFS node or some online public gateway. But the idea is for users to host a node themselves. > > It strikes me that this borders on incorporating business logic within > lavf. A user could achieve the same thing with a small shell script. > For example adding an alias that inspects calls to ffmpeg and sed:s > ipfs:// URLs accordingly > That might work but isn't really user friendly. It also doesn't help for tools/applications that incorporate ffmpeg to potentially use IPFS resources. KODI (when IPFS is merged into ffmpeg) is one such application where I'll be adding support for IPFS. But there are more that could potentially benefit. Think for example of OBS studio and blender (i haven't been in contact with them, all i know is that they use ffmpeg). > > > > > Question though. In a V2 patch, would it make sense to squash > > everything in > > one commit? > > Just look at what other patch series look like. This is small enough > that a single patch is likely enough yes > > /Tomas > > _______________________________________________ > 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". > _______________________________________________ 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".
next prev parent reply other threads:[~2022-01-31 22:01 UTC|newest] Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-01-31 13:51 Mark Gaiser 2022-01-31 13:51 ` [FFmpeg-devel] [PATCH 1/5] Early version of IPFS " Mark Gaiser 2022-01-31 15:59 ` Michael Niedermayer 2022-01-31 16:06 ` James Almer 2022-01-31 16:34 ` Mark Gaiser 2022-01-31 20:26 ` Lynne 2022-01-31 22:04 ` Mark Gaiser 2022-01-31 13:51 ` [FFmpeg-devel] [PATCH 2/5] Fix up IPNS support Mark Gaiser 2022-01-31 16:00 ` Michael Niedermayer 2022-01-31 13:51 ` [FFmpeg-devel] [PATCH 3/5] Merge IPNS and IPFS handling Mark Gaiser 2022-01-31 13:51 ` [FFmpeg-devel] [PATCH 4/5] Implement logic to determine the IPFS gateway Mark Gaiser 2022-01-31 13:51 ` [FFmpeg-devel] [PATCH 5/5] Fix review feedback Mark Gaiser 2022-01-31 15:46 ` Michael Niedermayer 2022-01-31 16:33 ` Mark Gaiser 2022-01-31 15:52 ` [FFmpeg-devel] [PATCH 0/5] Add IPFS and IPNS protocol support Tomas Härdin 2022-01-31 16:31 ` Mark Gaiser 2022-01-31 20:22 ` Tomas Härdin 2022-01-31 22:00 ` Mark Gaiser [this message] 2022-02-01 16:39 ` Tomas Härdin 2022-02-01 21:18 ` Mark Gaiser 2022-02-02 12:51 ` Tomas Härdin 2022-02-02 13:32 ` Mark Gaiser 2022-02-01 10:06 ` Michael Niedermayer 2022-02-01 16:43 ` Tomas Härdin 2022-02-02 13:48 ` Michael Niedermayer 2022-02-04 10:28 ` Tomas Härdin
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=CAPd6JnHcMZajgdh7Ayo5BwsW8G6N7namQj0RwvxLzGv1H4io8g@mail.gmail.com \ --to=markg85@gmail.com \ --cc=ffmpeg-devel@ffmpeg.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel This inbox may be cloned and mirrored by anyone: git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git # If you have public-inbox 1.1+ installed, you may # initialize and index your mirror using the following commands: public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \ ffmpegdev@gitmailbox.com public-inbox-index ffmpegdev Example config snippet for mirrors. AGPL code for this site: git clone https://public-inbox.org/public-inbox.git