linux-trace-devel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Steven Rostedt <rostedt@goodmis.org>
To: Tzvetomir Stoyanov <tz.stoyanov@gmail.com>
Cc: Ian Rogers <irogers@google.com>,
	Yordan Karadzhov <y.karadz@gmail.com>,
	Linux Trace Devel <linux-trace-devel@vger.kernel.org>
Subject: Re: [RFC PATCH 0/3] trace-cruncher: Initial support for perf
Date: Fri, 25 Feb 2022 09:06:11 -0500	[thread overview]
Message-ID: <20220225090611.66ca11e1@gandalf.local.home> (raw)
In-Reply-To: <CAPpZLN4RoEjhKJb+d2UB3cnJfuNzdSiu4dV8RZe=aCBUaBxWog@mail.gmail.com>

On Fri, 25 Feb 2022 12:21:22 +0200
Tzvetomir Stoyanov <tz.stoyanov@gmail.com> wrote:

> > The comment from the bug:
> > perf can link against libbfd if available, but the result is
> > undistributable as they are licenced under GPL v2 and v3+
> > respectively.
> >  
> 
> Hi Ian, thank you for this note. Trace-cruncher is a library licensed
> under LGPLv2.1, so it should not be a problem to use libbfd. Is my
> understanding correct, as I'm not an expert in all these licences ?

trace-cruncher can link with libbfd as LGPLv2.1 is compatible with GPLv3.
But note, this will limit what can link with trace-cruncher if it is
distributed and provides a library.

That is, if you have a trace-cruncher library, even though it is LGPLv2.1,
because it links to a GPLv3 library, then anything that links to the
trace-cruncher library must be GPLv3 compatible.

This is why I separated some code out of trace-cmd (although that code has
since been removed). I took GPL code, and used it in the binary of
trace-cmd. But I kept the libtracecmd code separate, where as, the
distributed libtracecmd code (that is LGPLv2.1) never contained any GPL
code, nor did it depend on any.

And now thinking about this more, and that we are starting to include
libraries into libtracecmd (for compression and other things), we need to
look carefully at the licenses for those libraries to make sure they do not
impose any more restrictions than what LGPL has.

The general rule of thumb is that you are constrained by the strictest
rules of everything provided by a library. If any part of the library
imposes a restriction, whatever links to that library must follow that
restriction.

-- Steve

      reply	other threads:[~2022-02-25 14:06 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-02-24 16:37 [RFC PATCH 0/3] trace-cruncher: Initial support for perf Tzvetomir Stoyanov (VMware)
2022-02-24 16:37 ` [RFC PATCH 1/3] trace-cruncher: Logic for resolving address to function name Tzvetomir Stoyanov (VMware)
2022-02-24 16:37 ` [RFC PATCH 2/3] trace-cruncher: Support for perf Tzvetomir Stoyanov (VMware)
2022-02-25 14:51   ` Arnaldo Carvalho de Melo
2022-02-25 15:38     ` Tzvetomir Stoyanov
2022-02-24 16:37 ` [RFC PATCH 3/3] trace-cruncher: perf example Tzvetomir Stoyanov (VMware)
2022-03-18  9:52   ` Yordan Karadzhov
2022-02-24 16:52 ` [RFC PATCH 0/3] trace-cruncher: Initial support for perf Ian Rogers
2022-02-25 10:21   ` Tzvetomir Stoyanov
2022-02-25 14:06     ` Steven Rostedt [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220225090611.66ca11e1@gandalf.local.home \
    --to=rostedt@goodmis.org \
    --cc=irogers@google.com \
    --cc=linux-trace-devel@vger.kernel.org \
    --cc=tz.stoyanov@gmail.com \
    --cc=y.karadz@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).