Re: [OE-core] [PATCH v3] systemd: fix remaining issues with statx and glibc 2.28+
Hi Chen, Will you pick up this fix? Still don't see it in the master-next Thanks, Ruslan On 12/22/18 1:48 AM, Khem Raj wrote: On Fri, Dec 21, 2018 at 11:45 AM Ruslan Bilovol wrote: Commit 776d14db9589 "systemd: Detect if statx struct is defined in sys/stat.h" did only a part of work, and systemd build fails on src/basic/chattr-util.c compilation due to same statx redefinition issue. Backport additional patch from systemd master LGTM Signed-off-by: Ruslan Bilovol --- v3: rebased to recent OE master (again) v2: rebased to recent OE master, added signoff ...-linux-stat.h-check-with-other-checks-and.patch | 78 ++ meta/recipes-core/systemd/systemd_239.bb | 1 + 2 files changed, 79 insertions(+) create mode 100644 meta/recipes-core/systemd/systemd/0001-meson-unify-linux-stat.h-check-with-other-checks-and.patch diff --git a/meta/recipes-core/systemd/systemd/0001-meson-unify-linux-stat.h-check-with-other-checks-and.patch b/meta/recipes-core/systemd/systemd/0001-meson-unify-linux-stat.h-check-with-other-checks-and.patch new file mode 100644 index 000..0db7fce --- /dev/null +++ b/meta/recipes-core/systemd/systemd/0001-meson-unify-linux-stat.h-check-with-other-checks-and.patch @@ -0,0 +1,78 @@ +From 9c869d08d82c73f62ab3527567858ce4b0cf1257 Mon Sep 17 00:00:00 2001 +From: =?UTF-8?q?Zbigniew=20J=C4=99drzejewski-Szmek?= +Date: Wed, 18 Jul 2018 17:26:17 +0200 +Subject: [PATCH] meson: unify linux/stat.h check with other checks and use + _GNU_SOURCE + +Using _GNU_SOURCE is better because that's how we include the headers in the +actual build, and some headers define different stuff when it is defined. +sys/stat.h for example defines 'struct statx' conditionally. + +Upstream-Status: Backport + +Signed-off-by: Ruslan Bilovol +--- + meson.build | 20 ++-- + src/basic/missing.h | 2 +- + 2 files changed, 15 insertions(+), 7 deletions(-) + +diff --git a/meson.build b/meson.build +index 68423bd..99035d2 100644 +--- a/meson.build b/meson.build +@@ -421,11 +421,9 @@ decl_headers = ''' + #include + #include + #include +-#include + #include + ''' + # FIXME: key_serial_t is only defined in keyutils.h, this is bound to fail +-# FIXME: these should use -D_GNU_SOURCE, since that is defined at build time + + foreach decl : ['char16_t', + 'char32_t', +@@ -436,13 +434,23 @@ foreach decl : ['char16_t', +] + + # We get -1 if the size cannot be determined +-have = cc.sizeof(decl, prefix : decl_headers) > 0 ++have = cc.sizeof(decl, prefix : decl_headers, args : '-D_GNU_SOURCE') > 0 ++ ++if decl == 'struct statx' ++if have ++want_linux_stat_h = false ++else ++have = cc.sizeof(decl, ++ prefix : decl_headers + '#include ', ++ args : '-D_GNU_SOURCE') > 0 ++want_linux_stat_h = have ++endif ++endif ++ + conf.set10('HAVE_' + decl.underscorify().to_upper(), have) + endforeach + +-conf.set10('HAVE_STRUCT_STATX_IN_SYS_STAT_H', cc.sizeof('struct statx', prefix : ''' +-#include +-''', args : '-D_GNU_SOURCE') > 0) ++conf.set10('WANT_LINUX_STAT_H', want_linux_stat_h) + + foreach decl : [['IFLA_INET6_ADDR_GEN_MODE', 'linux/if_link.h'], + ['IN6_ADDR_GEN_MODE_STABLE_PRIVACY', 'linux/if_link.h'], +diff --git a/src/basic/missing.h b/src/basic/missing.h +index 14ad3d4..9044683 100644 +--- a/src/basic/missing.h b/src/basic/missing.h +@@ -24,7 +24,7 @@ + #include + #include + +-#if !HAVE_STRUCT_STATX_IN_SYS_STAT_H ++#if WANT_LINUX_STAT_H + #include + #endif + +-- +1.9.1 + diff --git a/meta/recipes-core/systemd/systemd_239.bb b/meta/recipes-core/systemd/systemd_239.bb index 03acce2..91a026d 100644 --- a/meta/recipes-core/systemd/systemd_239.bb +++ b/meta/recipes-core/systemd/systemd_239.bb @@ -37,6 +37,7 @@ SRC_URI += "file://touchscreen.rules \ file://0001-timesync-changes-type-of-drift_freq-to-int64_t.patch \ file://0001-sysctl-Don-t-pass-null-directive-argument-to-s.patch \ file://0002-core-Fix-use-after-free-case-in-load_from_path.patch \ + file://0001-meson-unify-linux-stat.h-check-with-other-checks-and.patch \ " # patches made for musl are only applied on TCLIBC is musl -- 1.9.1 -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [PATCH v3] systemd: fix remaining issues with statx and glibc 2.28+
On Fri, Dec 21, 2018 at 11:45 AM Ruslan Bilovol wrote: > > Commit 776d14db9589 "systemd: Detect if statx struct is > defined in sys/stat.h" did only a part of work, and systemd > build fails on src/basic/chattr-util.c compilation due to > same statx redefinition issue. > Backport additional patch from systemd master > LGTM > Signed-off-by: Ruslan Bilovol > --- > > v3: rebased to recent OE master (again) > > v2: rebased to recent OE master, added signoff > > ...-linux-stat.h-check-with-other-checks-and.patch | 78 > ++ > meta/recipes-core/systemd/systemd_239.bb | 1 + > 2 files changed, 79 insertions(+) > create mode 100644 > meta/recipes-core/systemd/systemd/0001-meson-unify-linux-stat.h-check-with-other-checks-and.patch > > diff --git > a/meta/recipes-core/systemd/systemd/0001-meson-unify-linux-stat.h-check-with-other-checks-and.patch > > b/meta/recipes-core/systemd/systemd/0001-meson-unify-linux-stat.h-check-with-other-checks-and.patch > new file mode 100644 > index 000..0db7fce > --- /dev/null > +++ > b/meta/recipes-core/systemd/systemd/0001-meson-unify-linux-stat.h-check-with-other-checks-and.patch > @@ -0,0 +1,78 @@ > +From 9c869d08d82c73f62ab3527567858ce4b0cf1257 Mon Sep 17 00:00:00 2001 > +From: =?UTF-8?q?Zbigniew=20J=C4=99drzejewski-Szmek?= > +Date: Wed, 18 Jul 2018 17:26:17 +0200 > +Subject: [PATCH] meson: unify linux/stat.h check with other checks and use > + _GNU_SOURCE > + > +Using _GNU_SOURCE is better because that's how we include the headers in the > +actual build, and some headers define different stuff when it is defined. > +sys/stat.h for example defines 'struct statx' conditionally. > + > +Upstream-Status: Backport > + > +Signed-off-by: Ruslan Bilovol > +--- > + meson.build | 20 ++-- > + src/basic/missing.h | 2 +- > + 2 files changed, 15 insertions(+), 7 deletions(-) > + > +diff --git a/meson.build b/meson.build > +index 68423bd..99035d2 100644 > +--- a/meson.build > b/meson.build > +@@ -421,11 +421,9 @@ decl_headers = ''' > + #include > + #include > + #include > +-#include > + #include > + ''' > + # FIXME: key_serial_t is only defined in keyutils.h, this is bound to fail > +-# FIXME: these should use -D_GNU_SOURCE, since that is defined at build time > + > + foreach decl : ['char16_t', > + 'char32_t', > +@@ -436,13 +434,23 @@ foreach decl : ['char16_t', > +] > + > + # We get -1 if the size cannot be determined > +-have = cc.sizeof(decl, prefix : decl_headers) > 0 > ++have = cc.sizeof(decl, prefix : decl_headers, args : > '-D_GNU_SOURCE') > 0 > ++ > ++if decl == 'struct statx' > ++if have > ++want_linux_stat_h = false > ++else > ++have = cc.sizeof(decl, > ++ prefix : decl_headers + '#include > ', > ++ args : '-D_GNU_SOURCE') > 0 > ++want_linux_stat_h = have > ++endif > ++endif > ++ > + conf.set10('HAVE_' + decl.underscorify().to_upper(), have) > + endforeach > + > +-conf.set10('HAVE_STRUCT_STATX_IN_SYS_STAT_H', cc.sizeof('struct statx', > prefix : ''' > +-#include > +-''', args : '-D_GNU_SOURCE') > 0) > ++conf.set10('WANT_LINUX_STAT_H', want_linux_stat_h) > + > + foreach decl : [['IFLA_INET6_ADDR_GEN_MODE', 'linux/if_link.h'], > + ['IN6_ADDR_GEN_MODE_STABLE_PRIVACY', 'linux/if_link.h'], > +diff --git a/src/basic/missing.h b/src/basic/missing.h > +index 14ad3d4..9044683 100644 > +--- a/src/basic/missing.h > b/src/basic/missing.h > +@@ -24,7 +24,7 @@ > + #include > + #include > + > +-#if !HAVE_STRUCT_STATX_IN_SYS_STAT_H > ++#if WANT_LINUX_STAT_H > + #include > + #endif > + > +-- > +1.9.1 > + > diff --git a/meta/recipes-core/systemd/systemd_239.bb > b/meta/recipes-core/systemd/systemd_239.bb > index 03acce2..91a026d 100644 > --- a/meta/recipes-core/systemd/systemd_239.bb > +++ b/meta/recipes-core/systemd/systemd_239.bb > @@ -37,6 +37,7 @@ SRC_URI += "file://touchscreen.rules \ > file://0001-timesync-changes-type-of-drift_freq-to-int64_t.patch \ > file://0001-sysctl-Don-t-pass-null-directive-argument-to-s.patch \ > file://0002-core-Fix-use-after-free-case-in-load_from_path.patch \ > + > file://0001-meson-unify-linux-stat.h-check-with-other-checks-and.patch \ > " > > # patches made for musl are only applied on TCLIBC is musl > -- > 1.9.1 > -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
[OE-core] [PATCH v3] systemd: fix remaining issues with statx and glibc 2.28+
Commit 776d14db9589 "systemd: Detect if statx struct is defined in sys/stat.h" did only a part of work, and systemd build fails on src/basic/chattr-util.c compilation due to same statx redefinition issue. Backport additional patch from systemd master Signed-off-by: Ruslan Bilovol --- v3: rebased to recent OE master (again) v2: rebased to recent OE master, added signoff ...-linux-stat.h-check-with-other-checks-and.patch | 78 ++ meta/recipes-core/systemd/systemd_239.bb | 1 + 2 files changed, 79 insertions(+) create mode 100644 meta/recipes-core/systemd/systemd/0001-meson-unify-linux-stat.h-check-with-other-checks-and.patch diff --git a/meta/recipes-core/systemd/systemd/0001-meson-unify-linux-stat.h-check-with-other-checks-and.patch b/meta/recipes-core/systemd/systemd/0001-meson-unify-linux-stat.h-check-with-other-checks-and.patch new file mode 100644 index 000..0db7fce --- /dev/null +++ b/meta/recipes-core/systemd/systemd/0001-meson-unify-linux-stat.h-check-with-other-checks-and.patch @@ -0,0 +1,78 @@ +From 9c869d08d82c73f62ab3527567858ce4b0cf1257 Mon Sep 17 00:00:00 2001 +From: =?UTF-8?q?Zbigniew=20J=C4=99drzejewski-Szmek?= +Date: Wed, 18 Jul 2018 17:26:17 +0200 +Subject: [PATCH] meson: unify linux/stat.h check with other checks and use + _GNU_SOURCE + +Using _GNU_SOURCE is better because that's how we include the headers in the +actual build, and some headers define different stuff when it is defined. +sys/stat.h for example defines 'struct statx' conditionally. + +Upstream-Status: Backport + +Signed-off-by: Ruslan Bilovol +--- + meson.build | 20 ++-- + src/basic/missing.h | 2 +- + 2 files changed, 15 insertions(+), 7 deletions(-) + +diff --git a/meson.build b/meson.build +index 68423bd..99035d2 100644 +--- a/meson.build b/meson.build +@@ -421,11 +421,9 @@ decl_headers = ''' + #include + #include + #include +-#include + #include + ''' + # FIXME: key_serial_t is only defined in keyutils.h, this is bound to fail +-# FIXME: these should use -D_GNU_SOURCE, since that is defined at build time + + foreach decl : ['char16_t', + 'char32_t', +@@ -436,13 +434,23 @@ foreach decl : ['char16_t', +] + + # We get -1 if the size cannot be determined +-have = cc.sizeof(decl, prefix : decl_headers) > 0 ++have = cc.sizeof(decl, prefix : decl_headers, args : '-D_GNU_SOURCE') > 0 ++ ++if decl == 'struct statx' ++if have ++want_linux_stat_h = false ++else ++have = cc.sizeof(decl, ++ prefix : decl_headers + '#include ', ++ args : '-D_GNU_SOURCE') > 0 ++want_linux_stat_h = have ++endif ++endif ++ + conf.set10('HAVE_' + decl.underscorify().to_upper(), have) + endforeach + +-conf.set10('HAVE_STRUCT_STATX_IN_SYS_STAT_H', cc.sizeof('struct statx', prefix : ''' +-#include +-''', args : '-D_GNU_SOURCE') > 0) ++conf.set10('WANT_LINUX_STAT_H', want_linux_stat_h) + + foreach decl : [['IFLA_INET6_ADDR_GEN_MODE', 'linux/if_link.h'], + ['IN6_ADDR_GEN_MODE_STABLE_PRIVACY', 'linux/if_link.h'], +diff --git a/src/basic/missing.h b/src/basic/missing.h +index 14ad3d4..9044683 100644 +--- a/src/basic/missing.h b/src/basic/missing.h +@@ -24,7 +24,7 @@ + #include + #include + +-#if !HAVE_STRUCT_STATX_IN_SYS_STAT_H ++#if WANT_LINUX_STAT_H + #include + #endif + +-- +1.9.1 + diff --git a/meta/recipes-core/systemd/systemd_239.bb b/meta/recipes-core/systemd/systemd_239.bb index 03acce2..91a026d 100644 --- a/meta/recipes-core/systemd/systemd_239.bb +++ b/meta/recipes-core/systemd/systemd_239.bb @@ -37,6 +37,7 @@ SRC_URI += "file://touchscreen.rules \ file://0001-timesync-changes-type-of-drift_freq-to-int64_t.patch \ file://0001-sysctl-Don-t-pass-null-directive-argument-to-s.patch \ file://0002-core-Fix-use-after-free-case-in-load_from_path.patch \ + file://0001-meson-unify-linux-stat.h-check-with-other-checks-and.patch \ " # patches made for musl are only applied on TCLIBC is musl -- 1.9.1 -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core