[FFmpeg-devel] [PATCH] forgejo/workflows: add initial rudimentary CI

[FFmpeg-devel] [PATCH] forgejo/workflows: add initial rudimentary CI

[Timo Rothenpieler]

It runs basic fate with no special dependencies enabled on x86_64 and
aarch64.
---
 .forgejo/workflows/test.yml | 39 +++++++++++++++++++++++++++++++++++++
 .gitignore                  |  1 +
 2 files changed, 40 insertions(+)
 create mode 100644 .forgejo/workflows/test.yml

diff --git a/.forgejo/workflows/test.yml b/.forgejo/workflows/test.yml
new file mode 100644
index 0000000000..f9e032b78c
--- /dev/null
+++ b/.forgejo/workflows/test.yml
@@ -0,0 +1,39 @@
+on:
+  push:
+    branches:
+      - master
+  pull_request:
+
+jobs:
+  run_fate:
+    strategy:
+      fail-fast: false
+      matrix:
+        runner: [linux-amd64,linux-aarch64]
+    runs-on: ${{ matrix.runner }}
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+      - name: Configure
+        run: ./configure
+      - name: Build
+        run: make -j$(nproc)
+      - name: Restore Cached Fate-Suite
+        uses: actions/cache/restore@v4
+        with:
+          path: fate-suite
+          key: fate-suite
+          restore-keys: |
+            fate-suite-
+      - name: Sync Fate-Suite
+        id: fate
+        run: |
+          make fate-rsync SAMPLES=$PWD/fate-suite
+          echo "hash=$(find fate-suite -type f | sha256sum | cut -d' ' -f1)" >> $GITHUB_OUTPUT
+      - name: Cache Fate-Suite
+        uses: actions/cache/save@v4
+        with:
+          path: fate-suite
+          key: fate-suite-${{ steps.fate.outputs.hash }}
+      - name: Run Fate
+        run: make fate SAMPLES=$PWD/fate-suite -j$(nproc)
diff --git a/.gitignore b/.gitignore
index 59c89da5e0..4aa49c52c7 100644
--- a/.gitignore
+++ b/.gitignore
@@ -45,3 +45,4 @@
 /libavcodec/vulkan/*.c
 /libavfilter/vulkan/*.c
 /.*/
+!/.forgejo/
-- 
2.49.0

_______________________________________________
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".

Re: [FFmpeg-devel] [PATCH] forgejo/workflows: add initial rudimentary CI

[Leo Izen]

On 7/21/25 12:37, Timo Rothenpieler wrote:
> It runs basic fate with no special dependencies enabled on x86_64 and
> aarch64.
> ---
>   .forgejo/workflows/test.yml | 39 +++++++++++++++++++++++++++++++++++++
>   .gitignore                  |  1 +
>   2 files changed, 40 insertions(+)
>   create mode 100644 .forgejo/workflows/test.yml
> 
> diff --git a/.forgejo/workflows/test.yml b/.forgejo/workflows/test.yml
> new file mode 100644
> index 0000000000..f9e032b78c
> --- /dev/null
> +++ b/.forgejo/workflows/test.yml
> @@ -0,0 +1,39 @@
> +on:
> +  push:
> +    branches:
> +      - master
> +  pull_request:
> +
> +jobs:
> +  run_fate:
> +    strategy:
> +      fail-fast: false
> +      matrix:
> +        runner: [linux-amd64,linux-aarch64]
> +    runs-on: ${{ matrix.runner }}
> +    steps:
> +      - name: Checkout
> +        uses: actions/checkout@v4
> +      - name: Configure
> +        run: ./configure
> +      - name: Build
> +        run: make -j$(nproc)
> +      - name: Restore Cached Fate-Suite
> +        uses: actions/cache/restore@v4
> +        with:
> +          path: fate-suite
> +          key: fate-suite
> +          restore-keys: |
> +            fate-suite-
> +      - name: Sync Fate-Suite
> +        id: fate
> +        run: |
> +          make fate-rsync SAMPLES=$PWD/fate-suite
> +          echo "hash=$(find fate-suite -type f | sha256sum | cut -d' ' -f1)" >> $GITHUB_OUTPUT

$GITHUB_OUTPUT is probably a copypaste issue.

> +      - name: Cache Fate-Suite
> +        uses: actions/cache/save@v4
> +        with:
> +          path: fate-suite
> +          key: fate-suite-${{ steps.fate.outputs.hash }}
> +      - name: Run Fate
> +        run: make fate SAMPLES=$PWD/fate-suite -j$(nproc)
> diff --git a/.gitignore b/.gitignore
> index 59c89da5e0..4aa49c52c7 100644
> --- a/.gitignore
> +++ b/.gitignore
> @@ -45,3 +45,4 @@
>   /libavcodec/vulkan/*.c
>   /libavfilter/vulkan/*.c
>   /.*/
> +!/.forgejo/

_______________________________________________
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".

Re: [FFmpeg-devel] [PATCH] forgejo/workflows: add initial rudimentary CI

[Timo Rothenpieler]

On 7/21/2025 7:02 PM, Leo Izen wrote:
> On 7/21/25 12:37, Timo Rothenpieler wrote:
>> It runs basic fate with no special dependencies enabled on x86_64 and
>> aarch64.
>> ---
>>   .forgejo/workflows/test.yml | 39 +++++++++++++++++++++++++++++++++++++
>>   .gitignore                  |  1 +
>>   2 files changed, 40 insertions(+)
>>   create mode 100644 .forgejo/workflows/test.yml
>>
>> diff --git a/.forgejo/workflows/test.yml b/.forgejo/workflows/test.yml
>> new file mode 100644
>> index 0000000000..f9e032b78c
>> --- /dev/null
>> +++ b/.forgejo/workflows/test.yml
>> @@ -0,0 +1,39 @@
>> +on:
>> +  push:
>> +    branches:
>> +      - master
>> +  pull_request:
>> +
>> +jobs:
>> +  run_fate:
>> +    strategy:
>> +      fail-fast: false
>> +      matrix:
>> +        runner: [linux-amd64,linux-aarch64]
>> +    runs-on: ${{ matrix.runner }}
>> +    steps:
>> +      - name: Checkout
>> +        uses: actions/checkout@v4
>> +      - name: Configure
>> +        run: ./configure
>> +      - name: Build
>> +        run: make -j$(nproc)
>> +      - name: Restore Cached Fate-Suite
>> +        uses: actions/cache/restore@v4
>> +        with:
>> +          path: fate-suite
>> +          key: fate-suite
>> +          restore-keys: |
>> +            fate-suite-
>> +      - name: Sync Fate-Suite
>> +        id: fate
>> +        run: |
>> +          make fate-rsync SAMPLES=$PWD/fate-suite
>> +          echo "hash=$(find fate-suite -type f | sha256sum | cut -d' 
>> ' -f1)" >> $GITHUB_OUTPUT
> 
> $GITHUB_OUTPUT is probably a copypaste issue.

Negative, Forgejo Runners are fully compatible with GHA, so this is how 
you do it.

>> +      - name: Cache Fate-Suite
>> +        uses: actions/cache/save@v4
>> +        with:
>> +          path: fate-suite
>> +          key: fate-suite-${{ steps.fate.outputs.hash }}
>> +      - name: Run Fate
>> +        run: make fate SAMPLES=$PWD/fate-suite -j$(nproc)
>> diff --git a/.gitignore b/.gitignore
>> index 59c89da5e0..4aa49c52c7 100644
>> --- a/.gitignore
>> +++ b/.gitignore
>> @@ -45,3 +45,4 @@
>>   /libavcodec/vulkan/*.c
>>   /libavfilter/vulkan/*.c
>>   /.*/
>> +!/.forgejo/
> 
> _______________________________________________
> 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".

Re: [FFmpeg-devel] [PATCH] forgejo/workflows: add initial rudimentary CI

[Timo Rothenpieler]

On 7/21/2025 7:44 PM, Timo Rothenpieler wrote:
> On 7/21/2025 7:02 PM, Leo Izen wrote:
>> On 7/21/25 12:37, Timo Rothenpieler wrote:
>>> It runs basic fate with no special dependencies enabled on x86_64 and
>>> aarch64.
>>> ---
>>>   .forgejo/workflows/test.yml | 39 +++++++++++++++++++++++++++++++++++++
>>>   .gitignore                  |  1 +
>>>   2 files changed, 40 insertions(+)
>>>   create mode 100644 .forgejo/workflows/test.yml
>>>
>>> diff --git a/.forgejo/workflows/test.yml b/.forgejo/workflows/test.yml
>>> new file mode 100644
>>> index 0000000000..f9e032b78c
>>> --- /dev/null
>>> +++ b/.forgejo/workflows/test.yml
>>> @@ -0,0 +1,39 @@
>>> +on:
>>> +  push:
>>> +    branches:
>>> +      - master
>>> +  pull_request:
>>> +
>>> +jobs:
>>> +  run_fate:
>>> +    strategy:
>>> +      fail-fast: false
>>> +      matrix:
>>> +        runner: [linux-amd64,linux-aarch64]
>>> +    runs-on: ${{ matrix.runner }}
>>> +    steps:
>>> +      - name: Checkout
>>> +        uses: actions/checkout@v4
>>> +      - name: Configure
>>> +        run: ./configure
>>> +      - name: Build
>>> +        run: make -j$(nproc)
>>> +      - name: Restore Cached Fate-Suite
>>> +        uses: actions/cache/restore@v4
>>> +        with:
>>> +          path: fate-suite
>>> +          key: fate-suite
>>> +          restore-keys: |
>>> +            fate-suite-
>>> +      - name: Sync Fate-Suite
>>> +        id: fate
>>> +        run: |
>>> +          make fate-rsync SAMPLES=$PWD/fate-suite
>>> +          echo "hash=$(find fate-suite -type f | sha256sum | cut -d' 
>>> ' -f1)" >> $GITHUB_OUTPUT
>>
>> $GITHUB_OUTPUT is probably a copypaste issue.
> 
> Negative, Forgejo Runners are fully compatible with GHA, so this is how 
> you do it.

By now, both FORGEJO_OUTPUT and GITHUB_OUTPUT work, so I think renaming 
this makes it look a bit nicer. Will amend locally.

>>> +      - name: Cache Fate-Suite
>>> +        uses: actions/cache/save@v4
>>> +        with:
>>> +          path: fate-suite
>>> +          key: fate-suite-${{ steps.fate.outputs.hash }}
>>> +      - name: Run Fate
>>> +        run: make fate SAMPLES=$PWD/fate-suite -j$(nproc)
>>> diff --git a/.gitignore b/.gitignore
>>> index 59c89da5e0..4aa49c52c7 100644
>>> --- a/.gitignore
>>> +++ b/.gitignore
>>> @@ -45,3 +45,4 @@
>>>   /libavcodec/vulkan/*.c
>>>   /libavfilter/vulkan/*.c
>>>   /.*/
>>> +!/.forgejo/
>>
>> _______________________________________________
>> 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".

_______________________________________________
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".

Re: [FFmpeg-devel] [PATCH] forgejo/workflows: add initial rudimentary CI

[Michael Niedermayer]

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

On Mon, Jul 21, 2025 at 06:37:06PM +0200, Timo Rothenpieler wrote:
> It runs basic fate with no special dependencies enabled on x86_64 and
> aarch64.
> ---
>  .forgejo/workflows/test.yml | 39 +++++++++++++++++++++++++++++++++++++
>  .gitignore                  |  1 +
>  2 files changed, 40 insertions(+)
>  create mode 100644 .forgejo/workflows/test.yml
> 
> diff --git a/.forgejo/workflows/test.yml b/.forgejo/workflows/test.yml
> new file mode 100644
> index 0000000000..f9e032b78c
> --- /dev/null
> +++ b/.forgejo/workflows/test.yml
> @@ -0,0 +1,39 @@
> +on:
> +  push:
> +    branches:
> +      - master
> +  pull_request:
> +
> +jobs:
> +  run_fate:
> +    strategy:
> +      fail-fast: false
> +      matrix:
> +        runner: [linux-amd64,linux-aarch64]
> +    runs-on: ${{ matrix.runner }}
> +    steps:
> +      - name: Checkout
> +        uses: actions/checkout@v4
> +      - name: Configure

> +        run: ./configure

If you want to maximize coverage and maximize speed:
dash ./configure
--enable-gpl
(--enable-nonfree)
--enable-version3
--cc='ccache gcc' (or clang)
--assert-level=2
--tempprefix=somebasepaththatcanbeusedforcreatingtemporaryfiles
--enable-whatever-is-insalled

no more comments from me, patch can be applied once everyone is happy

thx

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

It is dangerous to be right in matters on which the established authorities
are wrong. -- Voltaire

[-- 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".

Re: [FFmpeg-devel] [PATCH] forgejo/workflows: add initial rudimentary CI

[Scott Theisen]

On 2025/07/21 18:17, Michael Niedermayer wrote:
> On Mon, Jul 21, 2025 at 06:37:06PM +0200, Timo Rothenpieler wrote:
>> It runs basic fate with no special dependencies enabled on x86_64 and
>> aarch64.
>> ---
>>   .forgejo/workflows/test.yml | 39 +++++++++++++++++++++++++++++++++++++
>>   .gitignore                  |  1 +
>>   2 files changed, 40 insertions(+)
>>   create mode 100644 .forgejo/workflows/test.yml
>>
>> diff --git a/.forgejo/workflows/test.yml b/.forgejo/workflows/test.yml
>> new file mode 100644
>> index 0000000000..f9e032b78c
>> --- /dev/null
>> +++ b/.forgejo/workflows/test.yml
>> @@ -0,0 +1,39 @@
>> +on:
>> +  push:
>> +    branches:
>> +      - master
>> +  pull_request:
>> +
>> +jobs:
>> +  run_fate:
>> +    strategy:
>> +      fail-fast: false
>> +      matrix:
>> +        runner: [linux-amd64,linux-aarch64]
>> +    runs-on: ${{ matrix.runner }}
>> +    steps:
>> +      - name: Checkout
>> +        uses: actions/checkout@v4
>> +      - name: Configure
>> +        run: ./configure
> If you want to maximize coverage and maximize speed:
> dash ./configure
> --enable-gpl
> (--enable-nonfree)
> --enable-version3
> --cc='ccache gcc' (or clang)

ccache would also need a set of actions/cache steps and if the caches 
are scoped as GitHub's are, probably also another workflow to clean up 
the caches when a pull request is closed.

> --assert-level=2
> --tempprefix=somebasepaththatcanbeusedforcreatingtemporaryfiles
> --enable-whatever-is-insalled
>
> no more comments from me, patch can be applied once everyone is happy
>
> thx
>
> [...]
>
>
> _______________________________________________
> 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".

Re: [FFmpeg-devel] [PATCH] forgejo/workflows: add initial rudimentary CI

[Kacper Michajlow]

On Tue, 22 Jul 2025 at 00:40, Scott Theisen <scott.the.elm@gmail.com> wrote:

> On 2025/07/21 18:17, Michael Niedermayer wrote:
> > On Mon, Jul 21, 2025 at 06:37:06PM +0200, Timo Rothenpieler wrote:
> >> It runs basic fate with no special dependencies enabled on x86_64 and
> >> aarch64.
> >> ---
> >>   .forgejo/workflows/test.yml | 39 +++++++++++++++++++++++++++++++++++++
> >>   .gitignore                  |  1 +
> >>   2 files changed, 40 insertions(+)
> >>   create mode 100644 .forgejo/workflows/test.yml
> >>
> >> diff --git a/.forgejo/workflows/test.yml b/.forgejo/workflows/test.yml
> >> new file mode 100644
> >> index 0000000000..f9e032b78c
> >> --- /dev/null
> >> +++ b/.forgejo/workflows/test.yml
> >> @@ -0,0 +1,39 @@
> >> +on:
> >> +  push:
> >> +    branches:
> >> +      - master
> >> +  pull_request:
> >> +
> >> +jobs:
> >> +  run_fate:
> >> +    strategy:
> >> +      fail-fast: false
> >> +      matrix:
> >> +        runner: [linux-amd64,linux-aarch64]
> >> +    runs-on: ${{ matrix.runner }}
> >> +    steps:
> >> +      - name: Checkout
> >> +        uses: actions/checkout@v4
> >> +      - name: Configure
> >> +        run: ./configure
> > If you want to maximize coverage and maximize speed:
> > dash ./configure
> > --enable-gpl
> > (--enable-nonfree)
> > --enable-version3
> > --cc='ccache gcc' (or clang)
>
> ccache would also need a set of actions/cache steps and if the caches
> are scoped as GitHub's are, probably also another workflow to clean up
> the caches when a pull request is closed


I can help with that, I have a pretty neat setup for ccache on github.
Should be transferable here. Although I agree with Timo, ccache is a risk
if we start having some miscompilations or stale objects. It generally
works, but I wouldn't use it unless really needed. For slow/less important
configs it can be enabled, but I wouldn't enable it for all builds.

- Kacper
_______________________________________________
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".

Re: [FFmpeg-devel] [PATCH] forgejo/workflows: add initial rudimentary CI

[Timo Rothenpieler]

On 7/22/2025 12:17 AM, Michael Niedermayer wrote:
> On Mon, Jul 21, 2025 at 06:37:06PM +0200, Timo Rothenpieler wrote:
>> It runs basic fate with no special dependencies enabled on x86_64 and
>> aarch64.
>> ---
>>   .forgejo/workflows/test.yml | 39 +++++++++++++++++++++++++++++++++++++
>>   .gitignore                  |  1 +
>>   2 files changed, 40 insertions(+)
>>   create mode 100644 .forgejo/workflows/test.yml
>>
>> diff --git a/.forgejo/workflows/test.yml b/.forgejo/workflows/test.yml
>> new file mode 100644
>> index 0000000000..f9e032b78c
>> --- /dev/null
>> +++ b/.forgejo/workflows/test.yml
>> @@ -0,0 +1,39 @@
>> +on:
>> +  push:
>> +    branches:
>> +      - master
>> +  pull_request:
>> +
>> +jobs:
>> +  run_fate:
>> +    strategy:
>> +      fail-fast: false
>> +      matrix:
>> +        runner: [linux-amd64,linux-aarch64]
>> +    runs-on: ${{ matrix.runner }}
>> +    steps:
>> +      - name: Checkout
>> +        uses: actions/checkout@v4
>> +      - name: Configure
> 
>> +        run: ./configure
> 
> If you want to maximize coverage and maximize speed:
> dash ./configure
> --enable-gpl
> (--enable-nonfree)

The builds aren't ever distributed, so --enable-nonfree is a good call.

> --enable-version3
> --cc='ccache gcc' (or clang)

I don't fully trust ccache to not cause spurious issues.
Gentoo stopped accepting bug reports if ccache was involved and it 
wasn't reproduced without it.
The builds are speedy enough that a full build each time doesn't seem 
too horrible.

I'd rather just add more runners if we ever run into capacity problems, 
which I honestly don't see happening anytime soon.

> --assert-level=2
> --tempprefix=somebasepaththatcanbeusedforcreatingtemporaryfiles

If I understand this right, all it does is use a fixed prefix in /tmp 
instead of just calling mktemp?
I don't immediately see the benefit of that, speed wise.

> --enable-whatever-is-insalled

Yeah, I intend to use the images with tons of deps which I already 
build, either directly or by spinning off some FFmpeg specific fork of them.
And then we need to decide on what makes sense to enable on CI.

> no more comments from me, patch can be applied once everyone is happy
> 
> thx
> 
> [...]
> 
> 
> _______________________________________________
> 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".