[Buildroot] procps-ng: disable for musl libc
Markus Mayer
mmayer at broadcom.com
Wed Jun 3 04:12:19 UTC 2020
On Tue, 2 Jun 2020 at 20:54, Baruch Siach <baruch at tkos.co.il> wrote:
>
> Hi Markus,
>
> On Wed, Jun 03 2020, Markus Mayer wrote:
> > There is a patch called "procps-ng: disable for musl libc" that was
> > applied to 2018.02.x:
> >
> > https://patchwork.ozlabs.org/project/buildroot/patch/3f2a94307e6c380842726b8158e8ec2e9d24372a.1530616293.git.baruch@tkos.co.il/
> >
> > This patch does not seem to be in any of the following releases nor is
> > it in master.
> >
> > $ for i in 2018 2019 2020; do for j in 02 05 08 11;do echo "$i/$j";
> > git log origin/$i.$j.x | grep 'procps-ng: disable for musl libc';
> > done;done
> > 2018/02
> > procps-ng: disable for musl libc
> > 2018/05
> > 2018/08
> > 2018/11
> > 2019/02
> > 2019/05
> > 2019/08
> > 2019/11
> > 2020/02
> >
> > Is this on purpose or should this patch be applied to all releases
> > going forward?
>
> This is on purpose.
>
> The commit log you liked to above specifically mentions musl version
> 1.1.8 in 2018.02.x. Later Buildroot releases include newer versions of
> musl that implement the missing fopencookie().
[resending for the benefit of the list]
Interesting. Do I need to turn on anything to enable this feature? I
am still seeing the very same error with 2020.05 and musl.
CC procio.o
CC proc/alloc.lo
CC proc/devname.lo
CC proc/escape.lo
CC proc/numa.lo
procio.c:50:8: error: unknown type name 'cookie_io_functions_t'
static cookie_io_functions_t procio = {
^~~~~~~~~~~~~~~~~~~~~
procio.c:51:5: error: field name not in record or union initializer
.read = proc_read,
^
procio.c:51:5: note: (near initialization for 'procio')
procio.c:51:14: warning: initialization makes integer from pointer
without a cast [-Wint-conversion]
.read = proc_read,
^~~~~~~~~
procio.c:51:14: note: (near initialization for 'procio')
procio.c:51:14: error: initializer element is not computable at load time
procio.c:51:14: note: (near initialization for 'procio')
procio.c:52:5: error: field name not in record or union initializer
.write = proc_write,
^
procio.c:52:5: note: (near initialization for 'procio')
procio.c:52:14: warning: excess elements in scalar initializer
.write = proc_write,
^~~~~~~~~~
procio.c:52:14: note: (near initialization for 'procio')
procio.c:53:5: error: field name not in record or union initializer
.seek = NULL,
^
procio.c:53:5: note: (near initialization for 'procio')
procio.c:53:14: warning: excess elements in scalar initializer
.seek = NULL,
^~~~
procio.c:53:14: note: (near initialization for 'procio')
procio.c:54:5: error: field name not in record or union initializer
.close = proc_close,
^
procio.c:54:5: note: (near initialization for 'procio')
procio.c:54:14: warning: excess elements in scalar initializer
.close = proc_close,
^~~~~~~~~~
procio.c:54:14: note: (near initialization for 'procio')
procio.c: In function 'fprocopen':
procio.c:136:11: warning: implicit declaration of function
'fopencookie' [-Wimplicit-function-declaration]
handle = fopencookie(cookie, mode, procio);
^~~~~~~~~~~
procio.c:136:9: warning: assignment makes pointer from integer without
a cast [-Wint-conversion]
handle = fopencookie(cookie, mode, procio);
^
Makefile:1501: recipe for target 'procio.o' failed
It works fine with glibc. I just started experimenting with libmusl.
Thanks,
-Markus
More information about the buildroot
mailing list