Messages in this thread Patch in this message | | | Date | Tue, 15 Nov 2016 16:49:29 -0500 | From | Steven Rostedt <> | Subject | Re: [PATCH 1/2] trace-cmd: Makefile - link all trace-* objects into ctracecmd.so |
| |
On Mon, 14 Nov 2016 11:55:42 +0000 Andreas Platschek <andreas.platschek@opentech.at> wrote:
> Got a few undefined symbols when using the Python interface to trace-cmd, > the first one was: > > root@wheezy:~/trace-cmd# python tracecmd.py > Traceback (most recent call last): > File "tracecmd.py", line 22, in <module> > from ctracecmd import * > ImportError: /home/andi/working_git/trace-cmd/ctracecmd.so: undefined symbol: get_file_content > > linking the objects that contain those symbols fixes the problem. > > Signed-off-by: Andreas Platschek <andreas.platschek@opentech.at> > --- > Makefile | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/Makefile b/Makefile > index 62cb25b..f86cb77 100644 > --- a/Makefile > +++ b/Makefile > @@ -344,7 +344,8 @@ TCMD_LIB_OBJS = $(PEVENT_LIB_OBJS) trace-util.o trace-input.o trace-ftrace.o \ > trace-output.o trace-record.o trace-recorder.o \ > trace-restore.o trace-usage.o trace-blk-hack.o \ > kbuffer-parse.o event-plugin.o trace-hooks.o \ > - trace-msg.o > + trace-msg.o trace-stat.o trace-listen.o trace-stream.o \ > + trace-read.o trace-profile.o trace-hash.o
Actually, I can't do this. The files in the library are licensed under LGPL, but these files you added are all GPL.
There's a reason I separated them as such. I guess we may be able to move things out of the files if they are properly authored. That is, if I wrote them. Some code I took and used in trace-cmd came from GPL v2 code, so those must remain GPL v2.
-- Steve
> > PLUGIN_OBJS = > PLUGIN_OBJS += plugin_jbd2.o
| |