[oe] [meta-oe][PATCH 1/2] collectd: Fix build with glibc 2.30
Adrian Bunk
bunk at stusta.de
Sun Jul 28 16:15:09 UTC 2019
On Sun, Jul 28, 2019 at 08:26:18AM -0700, Khem Raj wrote:
>
> On 7/28/19 2:37 AM, Adrian Bunk wrote:
> > On Sat, Jul 27, 2019 at 01:06:13PM -0700, Khem Raj wrote:
> > > ...
> > > +Glibc 2.30 has added deprecation notice and collectd detects it as
> > > +warning
> > > +
> > > +Fixes
> > > +sys/sysctl.h:21:2: error: "The <sys/sysctl.h> header is deprecated and will be removed." [-Werror,-W#warnings]
> > > ...
> > This package accumulates patches in OE that could be avoided by
> > configuring with --disable-werror instead.
>
> thats true but it would mask these issues by disabling werror and we wont be
> doing submissions like https://github.com/collectd/collectd/pull/3234,
With what range of glibc versions has this submission been verified?
Putting !defined(__GLIBC__) into source files is usually wrong,
and in this case it is unclear what happens with ancient glibc versions.
Note that --disable-werror would do the right thing will all glibc
versions past, present and future since the file will no longer be
included without any patch needed once the header is actually gone.
> I would like us to fix things upstream instead of masking them.
Status quo is that people look at harmless issues like this one in the
tiny subset of packages where -Werror is used, but noone seems to care
about serious runtime problems like implicit function declarations in
other packages.
And looking at the huge number of questionable patches in OE I don't
agree that it would in general be a good idea to apply fixes for random
issues in OE before they have been applied upstream - there are plenty
examples in the open source world where downstreams "fixing" issues in
upstream software caused problems, including major security issues
(I remember Debian/Ubuntu releases shipping a patched openssl generating
predictable private keys due to a bogus Debian "fix" for a Valgrind warning).
Reality is often more complicated, but in general downstream
distributions should try to avoid touching the upstream code
and aim at shipping unpatched upstream sources whenever possible.
cu
Adrian
--
"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed
More information about the Openembedded-devel
mailing list