The subject is a little odd in the wording. Added is the past tense of the verb. Also, our patches usually include the name of the recipe file so that scanning through the list of commits becomes clearer what area the patch is touching. Finally, including the branch name in the commit message makes it harder to think of the patch being able to apply to other branches. I think a better subject would be:

packagegroup-arago-tisdk-multimedia: Add ffmpeg package

And as Chirag pointed out, unless a patch is directly targeting issues related to a specific branch, we usually include master as well to make it clear that the patch can/should be applied to both.

So:

[meta-arago][master/scarthgap][PATCH] ...


Please submit a v2.


On 7/22/2024 5:16 PM, Thorsten Lannynd via lists.yoctoproject.org wrote:
Added ffmpeg package to scarthgap

Signed-off-by: Thorsten Lannynd <[email protected]>
---
  .../packagegroups/packagegroup-arago-tisdk-multimedia.bb         | 1 +
  1 file changed, 1 insertion(+)

diff --git 
a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-multimedia.bb
 
b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-multimedia.bb
index 6571d484..2085ad09 100644
--- 
a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-multimedia.bb
+++ 
b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-multimedia.bb
@@ -8,6 +8,7 @@ inherit packagegroup
MULTIMEDIA = " \
      amsdk-av-files \
+    ffmpeg \
      libcamera \
      libcamera-gst \
  "

--
Ryan Eatmon                [email protected]
-----------------------------------------
Texas Instruments, Inc.  -  LCPD  -  MGTS


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#15465): 
https://lists.yoctoproject.org/g/meta-arago/message/15465
Mute This Topic: https://lists.yoctoproject.org/mt/107495232/21656
Group Owner: [email protected]
Unsubscribe: https://lists.yoctoproject.org/g/meta-arago/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to