[FFmpeg-devel] [PATCH] bugfix and split of source plugin list (PR #20225)

[michaelni]

PR #20225 opened by michaelni
URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20225
Patch URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20225.patch


From a489870a2b35bf0e5a6a9a12696bceb211e18a35 Mon Sep 17 00:00:00 2001
From: Michael Niedermayer <michael at niedermayer.cc>
Date: Mon, 11 Aug 2025 20:06:25 +0200
Subject: [PATCH 1/2] tools/merge-all-source-plugins: Fix merge_internal()
 return code

Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
---
 tools/merge-all-source-plugins | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/tools/merge-all-source-plugins b/tools/merge-all-source-plugins
index cd030cdabe..27f64f690c 100755
--- a/tools/merge-all-source-plugins
+++ b/tools/merge-all-source-plugins
@@ -2,8 +2,11 @@
 
 #If a version is set then we only try merging a source plugin with matching version as a generic one could change the ABI to master HEAD
 merge_internal(){ # $1=repository, $2=refspec
-    [ -n "$version" ] && git pull --no-rebase --log --stat --commit --no-edit  $1 sourceplugin-$2-$version
-    [ -z "$version" ] && git pull --no-rebase --log --stat --commit --no-edit  $1 sourceplugin-$2
+    if [ -n "$version" ] ; then
+        git pull --no-rebase --log --stat --commit --no-edit  $1 sourceplugin-$2-$version
+    else
+        git pull --no-rebase --log --stat --commit --no-edit  $1 sourceplugin-$2
+    fi
 }
 
 unset succeeded failed version
-- 
2.49.1


From b0480e9c25592c02c806974a2c0db93a172c5a7c Mon Sep 17 00:00:00 2001
From: Michael Niedermayer <michael at niedermayer.cc>
Date: Tue, 12 Aug 2025 12:43:06 +0200
Subject: [PATCH 2/2] tools: Split the list of source plugins out of
 "merge-all-source-plugins"

---
 tools/merge-all-source-plugins | 11 ++++++++++-
 tools/source-plugins.txt       |  4 ++++
 2 files changed, 14 insertions(+), 1 deletion(-)
 create mode 100644 tools/source-plugins.txt

diff --git a/tools/merge-all-source-plugins b/tools/merge-all-source-plugins
index 27f64f690c..575bbfbff9 100755
--- a/tools/merge-all-source-plugins
+++ b/tools/merge-all-source-plugins
@@ -34,7 +34,16 @@ git diff --cached --exit-code >/dev/null ||\
 
 #version="12.34"
 
-merge "https://github.com/michaelni/FFmpeg.git" "libpostproc"
+[ $# -ne 1 ] &&\
+    error "Usage: $0 source-plugins.txt"
+
+while IFS=' ' read -r a b; do
+    case "$a" in
+        ''|'#'*) continue ;;
+    esac
+
+    merge "$a" "$b"
+done < "$1"
 
 [ -n "$version"  ] && echo version: $version
 [ -n "$succeeded" ] && echo Succeeded merging: $succeeded
diff --git a/tools/source-plugins.txt b/tools/source-plugins.txt
new file mode 100644
index 0000000000..6940140ccc
--- /dev/null
+++ b/tools/source-plugins.txt
@@ -0,0 +1,4 @@
+# List of source plugins
+
+# libpostproc, GPL, maintained by Michael Niedermayer
+https://github.com/michaelni/FFmpeg.git libpostproc
-- 
2.49.1