[FFmpeg-devel] [PATCH] avformat/avio: Add Metacube support

[Derek Buitenhuis]

On 04/05/2021 18:22, Steinar H. Gunderson wrote:
> For a libavformat-using program that does its own transports? Sure, I'm
> already doing that in other projects. For anything that wants to use the
> ffurl transports -- and in particular, stream from the ffmpeg(1) client?
> I don't see how that would be possible, unless I'm misunderstanding what
> you're suggesting.

Yes that's what I meant. It could in theory be reworked to to be in avformat/ffurl
but not avio itself (just the generic mechanisms by which it works), though, too.


>>   * You need to provide documentation / links on, like, what 'metacube' *is*. It's
>>     absolutely unclear to anyone reading this patch and associated doxy what it is
>>     or what its usecase is. AFAICT it's your own project?
> 
> I believe this was already covered in the commit message...? But yes, Cubemap
> is my own project, and Metacube is its framing protocol.

Only sorta. It doesn't link to he project, and Google wasn't immediately helpful.
There's a bunch of information about implementation details and interactions, but
not a ton on high level things like: What is the usecase? Why do we want this feature?
What's the plan for how it'll be used? What's the end goal? It's all kind of nebulous
to me, but maybe I'm not bright enough to infer or understand these...

There's no information in the doxy at all for users of this API who may not be
familiar with what metacube is, where it's from, what what its usecase is - why
they may want to use this API feature.

> I'm not seeing a lot of love for this patch, so I won't be pursuing it any
> further -- I wanted to upstream it due to user requests, but it is small
> enough that it can live in my own tree pretty much indefinitely.

This is your perogative, so I won't argue.

I will say that it is unclear enough to me what the goals and uses are of
the patch that it came off a bit like upstreaming something purely for personal
use, even if that wasn't intended. Apologies.

Cheers,
- Derek