[PATCH] trace-cmd: Generate proper soname

[PATCH] trace-cmd: Generate proper soname

[Patrick McLean]

From: Patrick McLean <patrick.mclean@sony.com>

The previous soname patch was an earlier version that included the full
path to the build output as the soname (not sure how that happened).
This just uses the basename as the soname, so ld will be able to
properly find the library.
---
 scripts/utils.mk | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/scripts/utils.mk b/scripts/utils.mk
index 9eb127d..98d7b3c 100644
--- a/scripts/utils.mk
+++ b/scripts/utils.mk
@@ -54,7 +54,7 @@ do_build_static_lib =				\
 
 do_compile_shared_library =			\
 	($(print_shared_lib_compile)		\
-	$(CC) --shared $^ -Wl,-soname,$@ -o $@)
+	$(CC) --shared $^ -Wl,-soname,$(@F) -o $@)
 
 do_compile_plugin_obj =				\
 	($(print_plugin_obj_compile)		\
-- 
2.22.0

Re: [PATCH] trace-cmd: Generate proper soname

[Steven Rostedt]

On Wed, 24 Jul 2019 18:48:08 -0700
Patrick McLean <chutzpah@gentoo.org> wrote:

> From: Patrick McLean <patrick.mclean@sony.com>
> 
> The previous soname patch was an earlier version that included the full
> path to the build output as the soname (not sure how that happened).
> This just uses the basename as the soname, so ld will be able to
> properly find the library.

Ah, so now I need to release a 2.8.4?

Thanks, ;-)

I'll wait till Friday before adding a 2.8.4.

-- Steve

> ---
>  scripts/utils.mk | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/scripts/utils.mk b/scripts/utils.mk
> index 9eb127d..98d7b3c 100644
> --- a/scripts/utils.mk
> +++ b/scripts/utils.mk
> @@ -54,7 +54,7 @@ do_build_static_lib =				\
>  
>  do_compile_shared_library =			\
>  	($(print_shared_lib_compile)		\
> -	$(CC) --shared $^ -Wl,-soname,$@ -o $@)
> +	$(CC) --shared $^ -Wl,-soname,$(@F) -o $@)
>  
>  do_compile_plugin_obj =				\
>  	($(print_plugin_obj_compile)		\

Re: [PATCH] trace-cmd: Generate proper soname

[Steven Rostedt]

On Wed, 24 Jul 2019 18:48:08 -0700
Patrick McLean <chutzpah@gentoo.org> wrote:

> From: Patrick McLean <patrick.mclean@sony.com>
> 
> The previous soname patch was an earlier version that included the full
> path to the build output as the soname (not sure how that happened).
> This just uses the basename as the soname, so ld will be able to
> properly find the library.

Hi Patrick,

You forgot to add the Signed-off-by, can you reply to this with one.
Thanks. I can't add it without it.

-- Steve

> ---
>  scripts/utils.mk | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/scripts/utils.mk b/scripts/utils.mk
> index 9eb127d..98d7b3c 100644
> --- a/scripts/utils.mk
> +++ b/scripts/utils.mk
> @@ -54,7 +54,7 @@ do_build_static_lib =				\
>  
>  do_compile_shared_library =			\
>  	($(print_shared_lib_compile)		\
> -	$(CC) --shared $^ -Wl,-soname,$@ -o $@)
> +	$(CC) --shared $^ -Wl,-soname,$(@F) -o $@)
>  
>  do_compile_plugin_obj =				\
>  	($(print_plugin_obj_compile)		\

Re: [PATCH] trace-cmd: Generate proper soname

[Patrick McLean]

On Thu, 25 Jul 2019 08:49:32 -0400
Steven Rostedt <rostedt@goodmis.org> wrote:

> On Wed, 24 Jul 2019 18:48:08 -0700
> Patrick McLean <chutzpah@gentoo.org> wrote:
> 
> > From: Patrick McLean <patrick.mclean@sony.com>
> > 
> > The previous soname patch was an earlier version that included the
> > full path to the build output as the soname (not sure how that
> > happened). This just uses the basename as the soname, so ld will be
> > able to properly find the library.  
> 
> Hi Patrick,
> 
> You forgot to add the Signed-off-by, can you reply to this with one.
> Thanks. I can't add it without it.
> 
My apologies.

> -- Steve
> 

Signed-off-by: Patrick McLean <patrick.mclean@sony.com>
> > ---
> >  scripts/utils.mk | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/scripts/utils.mk b/scripts/utils.mk
> > index 9eb127d..98d7b3c 100644
> > --- a/scripts/utils.mk
> > +++ b/scripts/utils.mk
> > @@ -54,7 +54,7 @@ do_build_static_lib
> > =				\ 
> >  do_compile_shared_library =			\
> >  	($(print_shared_lib_compile)		\
> > -	$(CC) --shared $^ -Wl,-soname,$@ -o $@)
> > +	$(CC) --shared $^ -Wl,-soname,$(@F) -o $@)
> >  
> >  do_compile_plugin_obj =				\
> >  	($(print_plugin_obj_compile)		\  
>