diff mbox series

[meta-arago,scarthgap/master,v2,6/7] arago.conf: Remove unneeded LICENSE_FLAGS_ACCEPTED items

Message ID 20250121175231.27183-6-afd@ti.com
State Accepted
Delegated to: Ryan Eatmon
Headers show
Series [meta-arago,scarthgap/master,v2,1/7] arago.conf: Remove old and unused config flags | expand

Commit Message

Andrew Davis Jan. 21, 2025, 5:52 p.m. UTC
netperf is no longer marked as non-commercial (587fe5877 in meta-oe).

faad2 is still marked commercial and AAC is still covered under several
patents that may require royalties. Not sure why we decided to whitelist
this, the commit adding it has no message. So, what can be added without
explanation can be removed without explanation :)

gst-ffmpeg is no longer a package, remove this too.

Signed-off-by: Andrew Davis <afd@ti.com>
---

Changes for v2:
 - Explain each change better in commit message

 meta-arago-distro/conf/distro/arago.conf | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Denys Dmytriyenko Jan. 22, 2025, 10:24 p.m. UTC | #1
On Tue, Jan 21, 2025 at 11:52:30AM -0600, Andrew Davis via lists.yoctoproject.org wrote:
> netperf is no longer marked as non-commercial (587fe5877 in meta-oe).
> 
> faad2 is still marked commercial and AAC is still covered under several
> patents that may require royalties. Not sure why we decided to whitelist
> this, the commit adding it has no message. So, what can be added without
> explanation can be removed without explanation :)

Heh... IIRC, there was an unconditional build dependency back then, probably a 
mistake. If removing it doesn't break anything, then it should be removed.


> gst-ffmpeg is no longer a package, remove this too.
> 
> Signed-off-by: Andrew Davis <afd@ti.com>
> ---
> 
> Changes for v2:
>  - Explain each change better in commit message
> 
>  meta-arago-distro/conf/distro/arago.conf | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/meta-arago-distro/conf/distro/arago.conf b/meta-arago-distro/conf/distro/arago.conf
> index 9f51ac26..033b8fcd 100644
> --- a/meta-arago-distro/conf/distro/arago.conf
> +++ b/meta-arago-distro/conf/distro/arago.conf
> @@ -167,7 +167,7 @@ SKIP_META_VIRT_SANITY_CHECK = "1"
>  # disable by default the codecs in libav that causes issues with shipping an
>  # Arago based SDK. No need to prevent gst-ffmpeg from being used since we
>  # disable troublesome software.
> -LICENSE_FLAGS_ACCEPTED = "commercial_gst-ffmpeg commercial_gstreamer1.0-libav commercial_faad2 non-commercial_netperf"
> +LICENSE_FLAGS_ACCEPTED = "commercial_gstreamer1.0-libav"
>  
>  # Do not package own copy of perl into devkit, rely on host one
>  ASSUME_PROVIDED += "nativesdk-perl"
> -- 
> 2.39.2
diff mbox series

Patch

diff --git a/meta-arago-distro/conf/distro/arago.conf b/meta-arago-distro/conf/distro/arago.conf
index 9f51ac26..033b8fcd 100644
--- a/meta-arago-distro/conf/distro/arago.conf
+++ b/meta-arago-distro/conf/distro/arago.conf
@@ -167,7 +167,7 @@  SKIP_META_VIRT_SANITY_CHECK = "1"
 # disable by default the codecs in libav that causes issues with shipping an
 # Arago based SDK. No need to prevent gst-ffmpeg from being used since we
 # disable troublesome software.
-LICENSE_FLAGS_ACCEPTED = "commercial_gst-ffmpeg commercial_gstreamer1.0-libav commercial_faad2 non-commercial_netperf"
+LICENSE_FLAGS_ACCEPTED = "commercial_gstreamer1.0-libav"
 
 # Do not package own copy of perl into devkit, rely on host one
 ASSUME_PROVIDED += "nativesdk-perl"