Re: [spice-streaming-agent 3/6] build: Remove -fvisibility detection from configure.ac

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

 



On Fri, Mar 02, 2018 at 02:27:57AM -0500, Frediano Ziglio wrote:
> > 
> > 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.
> > 
> > Signed-off-by: Christophe Fergeau <cfergeau@xxxxxxxxxx>
> 
> Have you compiled proposed Snir plugin?
> I think we are going to use soon this.
> Not that is hard to add again.

Hmm, the plugin would use it in its CFLAGS/CXXFLAGS, or alternatively,
it could use a symbol file. I don't think the agent binary needs it (in
other words, I would keep the Makefile.am change).
For what it's worth, maybe I'd change the test to use
https://cgit.freedesktop.org/gstreamer/common/tree/m4/as-compiler-flag.m4

VISIBILITY_CFLAGS=""
AS_COMPILER_FLAG([-fvisibility=hidden], [VISIBILITY_CFLAGS="-fvisibility=hidden"])
AC_SUBST(VISIBILITY_CFLAGS)

VISIBILITY_CXXFLAGS=""
if test "x$HAVE_CXX" = "xyes"; then
  AS_CXX_COMPILER_FLAG([-fvisibility=hidden], [VISIBILITY_CXXFLAGS="-fvisibility=hidden"])
fi
AC_SUBST(VISIBILITY_CXXFLAGS)

Christophe

Attachment: signature.asc
Description: PGP signature

_______________________________________________
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]