[PATCH yocto-autobuilder-helper 1/2] auh-config: enable systemd only for glibc

[PATCH yocto-autobuilder-helper 1/2] auh-config: enable systemd only for glibc

[Alexander Kanavin]

It is not guaranteed to build or work otherwise, and causes
update failures in unrelated recipes when building for musl for instance.

Signed-off-by: Alexander Kanavin <alex.kanavin@gmail.com>
---
 scripts/auh-config/local.conf.append | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/scripts/auh-config/local.conf.append b/scripts/auh-config/local.conf.append
index b18590f..417d228 100644
--- a/scripts/auh-config/local.conf.append
+++ b/scripts/auh-config/local.conf.append
@@ -1,4 +1,5 @@
 
 INHERIT += "buildhistory"
 LICENSE_FLAGS_WHITELIST = "commercial"
-DISTRO_FEATURES_append = ' systemd pam'
+DISTRO_FEATURES_append_libc-glibc = ' systemd'
+DISTRO_FEATURES_append = ' pam'
-- 
2.31.1

[PATCH yocto-autobuilder-helper 2/2] auh-config: adjust to new override syntax

[Alexander Kanavin]

Signed-off-by: Alexander Kanavin <alex.kanavin@gmail.com>
---
 scripts/auh-config/local.conf.append | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/scripts/auh-config/local.conf.append b/scripts/auh-config/local.conf.append
index 417d228..25c062b 100644
--- a/scripts/auh-config/local.conf.append
+++ b/scripts/auh-config/local.conf.append
@@ -1,5 +1,5 @@
 
 INHERIT += "buildhistory"
 LICENSE_FLAGS_WHITELIST = "commercial"
-DISTRO_FEATURES_append_libc-glibc = ' systemd'
-DISTRO_FEATURES_append = ' pam'
+DISTRO_FEATURES:append:libc-glibc = ' systemd'
+DISTRO_FEATURES:append = ' pam'
-- 
2.31.1

Re: [yocto] [PATCH yocto-autobuilder-helper 1/2] auh-config: enable systemd only for glibc

[Khem Raj]

On 8/14/21 1:56 AM, Alexander Kanavin wrote:
> It is not guaranteed to build or work otherwise, and causes
> update failures in unrelated recipes when building for musl for instance.
> 
> Signed-off-by: Alexander Kanavin <alex.kanavin@gmail.com>
> ---
>   scripts/auh-config/local.conf.append | 3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/scripts/auh-config/local.conf.append b/scripts/auh-config/local.conf.append
> index b18590f..417d228 100644
> --- a/scripts/auh-config/local.conf.append
> +++ b/scripts/auh-config/local.conf.append
> @@ -1,4 +1,5 @@
>   
>   INHERIT += "buildhistory"
>   LICENSE_FLAGS_WHITELIST = "commercial"
> -DISTRO_FEATURES_append = ' systemd pam'
> +DISTRO_FEATURES_append_libc-glibc = ' systemd'
> +DISTRO_FEATURES_append = ' pam'
> 

Please report the problems to mailing list and we can try to help if 
they do not work with musl. We do support systemd/musl combination and 
try to keep it working
this will limit testing this combination.

> 
> 
> 
> 

Re: [yocto] [PATCH yocto-autobuilder-helper 1/2] auh-config: enable systemd only for glibc

[Alexander Kanavin]

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

On Sun, 15 Aug 2021 at 03:30, Khem Raj <raj.khem@gmail.com> wrote:

>
> Please report the problems to mailing list and we can try to help if
> they do not work with musl. We do support systemd/musl combination and
> try to keep it working
> this will limit testing this combination.
>

This change is prompted by the recent period where musl patches wouldn't
even apply to systemd, which broke half of AUH upgrade attempts.

You need to enable systemd/musl on the autobuilder matrix first, if testing
the feature is a concern.

Alex

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

Re: [yocto] [PATCH yocto-autobuilder-helper 1/2] auh-config: enable systemd only for glibc

[Khem Raj]

On Sun, Aug 15, 2021 at 6:56 AM Alexander Kanavin
<alex.kanavin@gmail.com> wrote:
>
> On Sun, 15 Aug 2021 at 03:30, Khem Raj <raj.khem@gmail.com> wrote:
>>
>>
>> Please report the problems to mailing list and we can try to help if
>> they do not work with musl. We do support systemd/musl combination and
>> try to keep it working
>> this will limit testing this combination.
>
>
> This change is prompted by the recent period where musl patches wouldn't even apply to systemd, which broke half of AUH upgrade attempts.
>

Interesting, outside AB we build extended world on musl and it works
well. systemd porters usually port the patches as needed for musl

> You need to enable systemd/musl on the autobuilder matrix first, if testing the feature is a concern.

yes, I think thats a fair point, I will bring this up.

>
> Alex