diff mbox series

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

Message ID 20240102180706.41264-1-yoann.congal@smile.fr
State New
Headers show
Series [yocto-autobuilder-helper,dunfell] config.json: Make meta-oe source mirror config wider coverage | expand

Commit Message

Yoann Congal Jan. 2, 2024, 6:07 p.m. UTC
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(-)

Comments

Martin Jansa Jan. 2, 2024, 6:10 p.m. UTC | #1
> 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]
> -=-=-=-=-=-=-=-=-=-=-=-
>
>
Yoann Congal Jan. 2, 2024, 8:02 p.m. UTC | #2
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>]
>     -=-=-=-=-=-=-=-=-=-=-=-
>
diff mbox series

Patch

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",