Re: [PATCH 29/66] tools lib traceevent: Use str_error_r()

From: Steven Rostedt
Date: Tue Jul 12 2016 - 19:25:43 EST


On Tue, 12 Jul 2016 20:14:24 -0300
Arnaldo Carvalho de Melo <acme@xxxxxxxxxx> wrote:

> Em Tue, Jul 12, 2016 at 07:11:08PM -0400, Steven Rostedt escreveu:
> > On Tue, 12 Jul 2016 19:40:04 -0300 Arnaldo Carvalho de Melo <acme@xxxxxxxxxx> wrote:
> > > To make it portable to non-glibc systems, that follow the XSI variant
> > > instead of the GNU specific one that gets in place when _GNU_SOURCE is
> > > defined.
>
> > > #include "event-parse.h"
> > > @@ -6131,12 +6132,7 @@ int pevent_strerror(struct pevent *pevent __maybe_unused,
> > > + str_error_r(errnum, buf, buflen);
> > > return 0;
> >
> > What library is used with this? When I port this over to trace-cmd
> > (which is still needed as I develop this there), it fails to build.
> > "undefined reference to str_error_r"
>
> tools/lib/str_error_r.c
>
> Forgot about the out of tree copy :-\
>

Yeah, we really need to make this into a real library. I haven't had
the time to do that. Hopefully in August I can talk with some people at
LinuxCon to see the best way to go about doing that.

Anyway, I may just take that file and port it to trace-cmd.

-- Steve