diff mbox series

[meta-arago,master/scarthgap] packagegroup-arago-tisdk-multimedia: Add mpv package

Message ID 20240801155113.12804-1-t-lannynd@ti.com
State Superseded
Delegated to: Ryan Eatmon
Headers show
Series [meta-arago,master/scarthgap] packagegroup-arago-tisdk-multimedia: Add mpv package | expand

Commit Message

Thorsten Lannynd Aug. 1, 2024, 3:51 p.m. UTC
From: Thorsten Lannynd <t-lannynd@ti.com>

Add mpv package

Signed-off-by: Thorsten Lannynd <t-lannynd@ti.com>
---
 .../packagegroups/packagegroup-arago-tisdk-multimedia.bb         | 1 +
 1 file changed, 1 insertion(+)

Comments

Ryan Eatmon Aug. 1, 2024, 6:09 p.m. UTC | #1
It is common practice to add a v2, v3, v4, etc... to the subject line 
when sending edits to a patch.  Like this:

[meta-arago][master/scarthgap][PATCH v2] 
packagegroup-arago-tisdk-multimedia: Add mpv package

And then in the patch after the --- to list the different versiosn and 
what all you changed.  Like this:


Add mpv package to allow for video playback and streaming of online videos

Signed-off-by: Thorsten Lannynd <t-lannynd@ti.com>
---
v2: Updated commit message

   .../packagegroups/packagegroup-arago-tisdk-multimedia.bb         | 1 +
   1 file changed, 1 insertion(+)


That way the edit comment does not make it into the actual commit 
message, but you can still explain what changed from the previous version.

Please send a v3 and fix the above.


On 8/1/2024 11:18 AM, Thorsten Lannynd via lists.yoctoproject.org wrote:
> [Edited Message Follows]
> [Reason: Updated commit message]
> 
> From: Thorsten Lannynd <t-lannynd@ti.com>
> 
> Add mpv package to allow for video playback and streaming of online videos
> 
> Signed-off-by: Thorsten Lannynd <t-lannynd@ti.com>
> ---
>   .../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..1ed1b8ab 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
> @@ -10,6 +10,7 @@ MULTIMEDIA = " \
>       amsdk-av-files \
>       libcamera \
>       libcamera-gst \
> +    mpv \
>   "
>   
>   MULTIMEDIA:omapl138 = ""
diff mbox series

Patch

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..1ed1b8ab 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
@@ -10,6 +10,7 @@  MULTIMEDIA = " \
     amsdk-av-files \
     libcamera \
     libcamera-gst \
+    mpv \
 "
 
 MULTIMEDIA:omapl138 = ""