On Sun, Mar 22, 2026, at 4:23 PM, Michael Niedermayer via ffmpeg-devel wrote:
> Hi Martin
>
> On Sat, Mar 21, 2026 at 11:15:06PM +0200, Martin Storsjö via ffmpeg-devel 
> wrote:
>> On Sat, 21 Mar 2026, Michael Niedermayer via ffmpeg-devel wrote:
>> 
>> > > Nothing requires a dependency on external infrastructure.
>> > 
>> > The imported tree clearly points users and contributors to external 
>> > infrastructure:
>> > 
>> > +tests/checkasm/ext/docs/getting-started.md:url = 
>> > https://code.videolan.org/videolan/checkasm.git
>> > +tests/checkasm/ext/docs/getting-started.md:git submodule add -b release 
>> > https://code.videolan.org/videolan/checkasm subprojects/checkasm
>> > +tests/checkasm/ext/docs/introduction.md:- [**GitLab 
>> > Repository**](https://code.videolan.org/videolan/checkasm)
>> > +tests/checkasm/ext/docs/introduction.md:- [**Issue 
>> > Tracker**](https://code.videolan.org/videolan/checkasm/issues)
>> > +tests/checkasm/ext/docs/introduction.md:- [VideoLAN 
>> > Homepage](https://www.videolan.org/)
>> > +tests/checkasm/ext/src/html_data/body.html:        This report was 
>> > created using the <a id="checkasm-version" 
>> > href="https://code.videolan.org/videolan/checkasm";>checkasm</a>
>> > 
>> > So even if you do not consider that a hard technical dependency, it is 
>> > still an integration centered around external infrastructure and external 
>> > project entry points.
>> 
>> So if you are bothered by those files,
>
> I am not concerned about those files merely existing.
> I am concerned about FFmpeg importing code in a form that points users 
> and developers away from FFmpeg infrastructure.
>
> Whether this is done as a git subtree, a linked libcheckasm library, or 
> some other mechanism is a secondary technical question.
> My main concern is that checkasm, if used by FFmpeg as part of our 
> testing infrastructure,
> should be hosted and maintained in a way that FFmpeg can control on 
> code.ffmpeg.org.

And do you agree that having a checkasm repo on FFmpeg forgejo would alleviate 
those concerns?

>
> Checkasm is important enough that we need to be able to fix bugs, 
> address security issues,
> and add features without depending on an externally managed project and 
> release cycle.
>
> That is also why I said that if there is an upstream outside FFmpeg,
> then it is important that merging changes in both directions is not 
> only theoretically possible,
> but actually practical and beneficial to both sides.
>
>
>> then we could import the code in the
>> form of a code drop, removing all these files, and just keeping the source
>> files (the src/ and include/ subdirectories). With a script for updating
>> that from upstream, it should be straightforward to use.
>
> A script can certainly be useful, but that depends on what exactly it does.
>
> For example, if it loses local changes, makes merges unnecessarily 
> painful,
> or creates a messy and unclear git history, then that would not be a 
> good solution.
> If it provides a clean and maintainable update path, then that is a 
> different matter.
>
>
> [...]
>
>> > * a clear and reviewed attribution/relicensing story
>> 
>> For what it's worth, I have reviewed the whole history of this split out
>> repo, from the start from the dav1d checkasm import, through all the
>> refactoring and libraryfication of it.
>
> That is useful to know.
>
> If you are saying that you have reviewed the authorship, copyright,
> and relicensing history and believe it is correct, then that addresses 
> part of my concern.
>
> I still think that, for something of this size, it would be good if 
> that attribution
> and relicensing story were documented clearly enough that others do not 
> have to rely only on private review of the full history.
>
> thx
>
> [...]
> -- 
> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
>
> The greatest way to live with honor in this world is to be what we pretend
> to be. -- Socrates
>
> _______________________________________________
> ffmpeg-devel mailing list -- [email protected]
> To unsubscribe send an email to [email protected]
>
> Attachments:
> * signature.asc
_______________________________________________
ffmpeg-devel mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to