[yocto-autobuilder-helper][dunfell][PATCH] config.json: Make meta-oe source mirror config wider coverage

[yocto-autobuilder-helper][dunfell][PATCH] config.json: Make meta-oe source mirror config wider coverage

[Yoann Congal]

Some recipes depend on DISTRO_FEATURES, commercial licensing or compiler
config like fortran. Set these things so that we get wider soruce mirror
coverage and fewer warnings.

'commercial' license usage is ok here since we're not building and then
shipping any binaries or using it, only mirroring the source code.

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
(cherry picked from commit 93d47af8abbc6558f9a098a0c7fa6c0e901424a9)
Signed-off-by: Yoann Congal <yoann.congal@smile.fr>
Backport: new->old override syntax
---
This should fix the warnings of the AB meta-oe-mirror/dunfell builds
---
 config.json | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/config.json b/config.json
index 3520e9a..1a92c80 100644
--- a/config.json
+++ b/config.json
@@ -1100,7 +1100,11 @@
             ],
             "extravars" : [ 
                 "PNBLACKLIST[dvb-apps] = '1'",
-                "PNBLACKLIST[tvheadend] = '1'"
+                "PNBLACKLIST[tvheadend] = '1'",
+                "LICENSE_FLAGS_ACCEPTED = 'commercial'",
+                "DISTRO_FEATURES_append = ' pam systemd usrmerge'",
+                "FORTRAN_forcevariable = ',fortran'",
+                "RUNTIMETARGET_append_pn-gcc-runtime = ' libquadmath'"
             ],
             "step1" : {
                 "shortname" : "Sources pre-fetching",
-- 
2.39.2

Re: [yocto] [yocto-autobuilder-helper][dunfell][PATCH] config.json: Make meta-oe source mirror config wider coverage

[Martin Jansa]

[-- Attachment #1: Type: text/plain, Size: 2148 bytes --]

> Backport: new->old override syntax

FWIW: dunfell is forward compatible with new syntax, so this shouldn't be
needed (other than the consistency with other metadata).

On Tue, Jan 2, 2024 at 7:07 PM Yoann Congal <yoann.congal@smile.fr> wrote:

> Some recipes depend on DISTRO_FEATURES, commercial licensing or compiler
> config like fortran. Set these things so that we get wider soruce mirror
> coverage and fewer warnings.
>
> 'commercial' license usage is ok here since we're not building and then
> shipping any binaries or using it, only mirroring the source code.
>
> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> (cherry picked from commit 93d47af8abbc6558f9a098a0c7fa6c0e901424a9)
> Signed-off-by: Yoann Congal <yoann.congal@smile.fr>
> Backport: new->old override syntax
> ---
> This should fix the warnings of the AB meta-oe-mirror/dunfell builds
> ---
>  config.json | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/config.json b/config.json
> index 3520e9a..1a92c80 100644
> --- a/config.json
> +++ b/config.json
> @@ -1100,7 +1100,11 @@
>              ],
>              "extravars" : [
>                  "PNBLACKLIST[dvb-apps] = '1'",
> -                "PNBLACKLIST[tvheadend] = '1'"
> +                "PNBLACKLIST[tvheadend] = '1'",
> +                "LICENSE_FLAGS_ACCEPTED = 'commercial'",
> +                "DISTRO_FEATURES_append = ' pam systemd usrmerge'",
> +                "FORTRAN_forcevariable = ',fortran'",
> +                "RUNTIMETARGET_append_pn-gcc-runtime = ' libquadmath'"
>              ],
>              "step1" : {
>                  "shortname" : "Sources pre-fetching",
> --
> 2.39.2
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#62066):
> https://lists.yoctoproject.org/g/yocto/message/62066
> Mute This Topic: https://lists.yoctoproject.org/mt/103486578/3617156
> Group Owner: yocto+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/yocto/unsub [
> martin.jansa@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
>

[-- Attachment #2: Type: text/html, Size: 3391 bytes --]

Re: [yocto] [yocto-autobuilder-helper][dunfell][PATCH] config.json: Make meta-oe source mirror config wider coverage

[Yoann Congal]

Le 02/01/2024 à 19:10, Martin Jansa a écrit :
>> Backport: new->old override syntax
> 
> FWIW: dunfell is forward compatible with new syntax, so this shouldn't be needed (other than the consistency with other metadata).

I did not know that. Thanks!
Either way, I would use the old syntax in this case to match what is documented in local.conf template, recipe comments, etc. But it is nice to know :)

> On Tue, Jan 2, 2024 at 7:07 PM Yoann Congal <yoann.congal@smile.fr <mailto:yoann.congal@smile.fr>> wrote:
> 
>     Some recipes depend on DISTRO_FEATURES, commercial licensing or compiler
>     config like fortran. Set these things so that we get wider soruce mirror
>     coverage and fewer warnings.
> 
>     'commercial' license usage is ok here since we're not building and then
>     shipping any binaries or using it, only mirroring the source code.
> 
>     Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org <mailto:richard.purdie@linuxfoundation.org>>
>     (cherry picked from commit 93d47af8abbc6558f9a098a0c7fa6c0e901424a9)
>     Signed-off-by: Yoann Congal <yoann.congal@smile.fr <mailto:yoann.congal@smile.fr>>
>     Backport: new->old override syntax
>     ---
>     This should fix the warnings of the AB meta-oe-mirror/dunfell builds
>     ---
>      config.json | 6 +++++-
>      1 file changed, 5 insertions(+), 1 deletion(-)
> 
>     diff --git a/config.json b/config.json
>     index 3520e9a..1a92c80 100644
>     --- a/config.json
>     +++ b/config.json
>     @@ -1100,7 +1100,11 @@
>                  ],
>                  "extravars" : [
>                      "PNBLACKLIST[dvb-apps] = '1'",
>     -                "PNBLACKLIST[tvheadend] = '1'"
>     +                "PNBLACKLIST[tvheadend] = '1'",
>     +                "LICENSE_FLAGS_ACCEPTED = 'commercial'",
>     +                "DISTRO_FEATURES_append = ' pam systemd usrmerge'",
>     +                "FORTRAN_forcevariable = ',fortran'",
>     +                "RUNTIMETARGET_append_pn-gcc-runtime = ' libquadmath'"
>                  ],
>                  "step1" : {
>                      "shortname" : "Sources pre-fetching",
>     -- 
>     2.39.2
> 
> 
>     -=-=-=-=-=-=-=-=-=-=-=-
>     Links: You receive all messages sent to this group.
>     View/Reply Online (#62066): https://lists.yoctoproject.org/g/yocto/message/62066 <https://lists.yoctoproject.org/g/yocto/message/62066>
>     Mute This Topic: https://lists.yoctoproject.org/mt/103486578/3617156 <https://lists.yoctoproject.org/mt/103486578/3617156>
>     Group Owner: yocto+owner@lists.yoctoproject.org <mailto:yocto%2Bowner@lists.yoctoproject.org>
>     Unsubscribe: https://lists.yoctoproject.org/g/yocto/unsub <https://lists.yoctoproject.org/g/yocto/unsub> [martin.jansa@gmail.com <mailto:martin.jansa@gmail.com>]
>     -=-=-=-=-=-=-=-=-=-=-=-
> 

-- 
Yoann Congal
Smile ECS - Tech Expert