Re: [spice-streaming-agent v2] build: Don't use -fvisibility when building the agent

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> 
> In my testing (x86_64/gcc), this had no impact on the resulting binary,
> building with/without it gives the same stripped binary save for its
> buildid.
> The check for -fvisibility in configure.ac is kept as this is going to
> be useful when we ship dlopen'ed plugins.
> 
> Signed-off-by: Christophe Fergeau <cfergeau@xxxxxxxxxx>
> ---
> Here is what the patch would look like if we keep the configure.ac check
> for future usage.
> 
> Christophe
> 
> 
>  src/Makefile.am | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/src/Makefile.am b/src/Makefile.am
> index 3717b5c..857d763 100644
> --- a/src/Makefile.am
> +++ b/src/Makefile.am
> @@ -16,12 +16,10 @@ AM_CPPFLAGS = \
>  	$(NULL)
>  
>  AM_CFLAGS = \
> -	$(VISIBILITY_HIDDEN_CFLAGS) \
>  	$(WARN_CFLAGS) \
>  	$(NULL)
>  
>  AM_CXXFLAGS = \
> -	$(VISIBILITY_HIDDEN_CFLAGS) \
>  	$(WARN_CXXFLAGS) \
>  	$(NULL)
>  

Acked-by: Frediano Ziglio <fziglio@xxxxxxxxxx>

Not sure if with -fPIE and x86 this won't change slightly.

Frediano
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]