* [PATCH 0/2] (Trivial) README fix and libtracecmd SONAME
@ 2022-01-20 11:34 Dario Faggioli
2022-01-20 11:34 ` [PATCH 1/2] README: remove stale mention to libparsevent Dario Faggioli
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Dario Faggioli @ 2022-01-20 11:34 UTC (permalink / raw)
To: linux-trace-devel
Cc: Tzvetomir Stoyanov, Steven Rostedt, Steven Rostedt,
Tzvetomir Stoyanov (VMware)
Hi!
So, patch 1 of this small series just get rid of a stale mention to
libparsevent in the README file.
About patch 2. Some distributions have the requirement that the package
name contains/is related to the SONAME (for packages that ship libraries,
of course).
Currently, the SONAME of libtracecmd is something like libtracecmd.so.0.0.1
or libtracecmd.so.1.0.0. This is not a problem, but I think libtracecmd.so.0
(libtracecmd.so.1) is a bit more convenient.
It is also how things are for libtracefs and libtraceevent.
So, patch 2 does exactly that for libtracecmd, if that could be
interesting. :-)
If, instead, it was intentional and there is an actual reason for the
SONAME of libtracecmd to be the entire file name of the .so file, then I
guess I've missed it. In which case, sorry for the noise.
Regards
---
Dario Faggioli (2):
README: remove stale mention to libparsevent
libtracecmd: use major version as SONAME
lib/trace-cmd/Makefile | 2 +-
scripts/utils.mk | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
--
Dario Faggioli, Ph.D
http://about.me/dario.faggioli
Virtualization Software Engineer
SUSE Labs, SUSE https://www.suse.com/
-------------------------------------------------------------------
<<This happens because _I_ choose it to happen!>> (Raistlin Majere)
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH 1/2] README: remove stale mention to libparsevent
2022-01-20 11:34 [PATCH 0/2] (Trivial) README fix and libtracecmd SONAME Dario Faggioli
@ 2022-01-20 11:34 ` Dario Faggioli
2022-01-20 11:34 ` [PATCH 2/2] libtracecmd: use major version as SONAME Dario Faggioli
2022-01-23 2:08 ` [PATCH 0/2] (Trivial) README fix and libtracecmd SONAME Steven Rostedt
2 siblings, 0 replies; 4+ messages in thread
From: Dario Faggioli @ 2022-01-20 11:34 UTC (permalink / raw)
To: linux-trace-devel; +Cc: Tzvetomir Stoyanov (VMware), Steven Rostedt
Signed-off-by: Dario Faggioli <dfaggioli@suse.com>
---
Cc: Tzvetomir Stoyanov (VMware) <tz.stoyanov@gmail.com>
Cc: Steven Rostedt <rostedt@goodmis.org>
---
README | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/README b/README
index 9a3533d..9d8c127 100644
--- a/README
+++ b/README
@@ -14,9 +14,7 @@ These files make up the code that create the trace-cmd programs.
This includes the GUI interface application kernelshark as well
as trace-graph and trace-view.
-These files also make up the code to create the libraries
-
- libtracecmd and libparsevent
+These files also make up the code to create the libtracecmd library.
The applications are licensed under the GNU General Public License 2.0
(see COPYING) and the libraries are licensed under the GNU
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH 2/2] libtracecmd: use major version as SONAME
2022-01-20 11:34 [PATCH 0/2] (Trivial) README fix and libtracecmd SONAME Dario Faggioli
2022-01-20 11:34 ` [PATCH 1/2] README: remove stale mention to libparsevent Dario Faggioli
@ 2022-01-20 11:34 ` Dario Faggioli
2022-01-23 2:08 ` [PATCH 0/2] (Trivial) README fix and libtracecmd SONAME Steven Rostedt
2 siblings, 0 replies; 4+ messages in thread
From: Dario Faggioli @ 2022-01-20 11:34 UTC (permalink / raw)
To: linux-trace-devel; +Cc: Tzvetomir Stoyanov (VMware), Steven Rostedt
Currently, the SONAME of libtracecmd is libtracecmd.so.0.0.1, which
is legit and overall fine, but slightly inconvenient for packaging.
For other libreries, like libtraceevent and libtracefs, it is set to
libtraceevent.so.1 and libtracefs.so.1. Do the same here
Signed-off-by: Dario Faggioli <dfaggioli@suse.com>
---
Cc: Tzvetomir Stoyanov (VMware) <tz.stoyanov@gmail.com>
Cc: Steven Rostedt <rostedt@goodmis.org>
---
lib/trace-cmd/Makefile | 2 +-
scripts/utils.mk | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/lib/trace-cmd/Makefile b/lib/trace-cmd/Makefile
index 1760031..c9d36d1 100644
--- a/lib/trace-cmd/Makefile
+++ b/lib/trace-cmd/Makefile
@@ -55,7 +55,7 @@ $(LIBTRACECMD_SHARED_SO): $(LIBTRACECMD_SHARED_VERSION)
libtracecmd.so: force $(LIBTRACECMD_SHARED_SO)
$(LIBTRACECMD_SHARED): $(OBJS)
- $(Q)$(call do_compile_shared_library)
+ $(Q)$(call do_compile_shared_library,$(notdir $(LIBTRACECMD_SHARED_VERSION)))
$(bdir)/%.o: %.c
$(Q)$(call do_fpic_compile)
diff --git a/scripts/utils.mk b/scripts/utils.mk
index 53b2b07..3fc2d74 100644
--- a/scripts/utils.mk
+++ b/scripts/utils.mk
@@ -72,7 +72,7 @@ do_build_static_lib = \
do_compile_shared_library = \
($(print_shared_lib_compile) \
- $(CC) --shared $^ '-Wl,-soname,$(@F),-rpath=$$ORIGIN' -o $@ $(LDFLAGS) $(LIBS))
+ $(CC) --shared $^ '-Wl,-soname,$(1),-rpath=$$ORIGIN' -o $@ $(LDFLAGS) $(LIBS))
do_compile_plugin_obj = \
($(print_plugin_obj_compile) \
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH 0/2] (Trivial) README fix and libtracecmd SONAME
2022-01-20 11:34 [PATCH 0/2] (Trivial) README fix and libtracecmd SONAME Dario Faggioli
2022-01-20 11:34 ` [PATCH 1/2] README: remove stale mention to libparsevent Dario Faggioli
2022-01-20 11:34 ` [PATCH 2/2] libtracecmd: use major version as SONAME Dario Faggioli
@ 2022-01-23 2:08 ` Steven Rostedt
2 siblings, 0 replies; 4+ messages in thread
From: Steven Rostedt @ 2022-01-23 2:08 UTC (permalink / raw)
To: Dario Faggioli; +Cc: linux-trace-devel, Tzvetomir Stoyanov
On Thu, 20 Jan 2022 12:34:08 +0100
Dario Faggioli <dfaggioli@suse.com> wrote:
> Hi!
>
> So, patch 1 of this small series just get rid of a stale mention to
> libparsevent in the README file.
>
> About patch 2. Some distributions have the requirement that the package
> name contains/is related to the SONAME (for packages that ship libraries,
> of course).
>
> Currently, the SONAME of libtracecmd is something like libtracecmd.so.0.0.1
> or libtracecmd.so.1.0.0. This is not a problem, but I think libtracecmd.so.0
> (libtracecmd.so.1) is a bit more convenient.
>
> It is also how things are for libtracefs and libtraceevent.
>
> So, patch 2 does exactly that for libtracecmd, if that could be
> interesting. :-)
>
> If, instead, it was intentional and there is an actual reason for the
> SONAME of libtracecmd to be the entire file name of the .so file, then I
> guess I've missed it. In which case, sorry for the noise.
>
>
Applied. Thanks Dario.
Although I had to tweak the subjects to have them start with a capital
letter (like a sentence). And I also fixed a typo in the change log of
the second patch.
-- Steve
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2022-01-23 2:08 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-01-20 11:34 [PATCH 0/2] (Trivial) README fix and libtracecmd SONAME Dario Faggioli
2022-01-20 11:34 ` [PATCH 1/2] README: remove stale mention to libparsevent Dario Faggioli
2022-01-20 11:34 ` [PATCH 2/2] libtracecmd: use major version as SONAME Dario Faggioli
2022-01-23 2:08 ` [PATCH 0/2] (Trivial) README fix and libtracecmd SONAME Steven Rostedt
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.