From 9842a6708d642da8e32eb26d5db319c0283ec068 Mon Sep 17 00:00:00 2001 From: Thibault Saunier Date: Thu, 20 Oct 2016 17:17:54 -0300 Subject: [PATCH] Revert "meson: Use the new `pic` argument on static libs" This reverts commit e3c22605ae96ee1747020c4f367d49faf6916e14. pic was added after 0.35 and will be present in 0.36 (meson documentation was wrong). --- gst-libs/gst/audio/meson.build | 15 ++++++--------- meson.build | 8 ++++++++ 2 files changed, 14 insertions(+), 9 deletions(-) diff --git a/gst-libs/gst/audio/meson.build b/gst-libs/gst/audio/meson.build index 987c3600e4..c68f113ad8 100644 --- a/gst-libs/gst/audio/meson.build +++ b/gst-libs/gst/audio/meson.build @@ -91,11 +91,10 @@ simd_dependencies = [] if have_sse audio_resampler_sse = static_library('audio_resampler_sse', ['audio-resampler-x86-sse.c', gstaudio_h], - c_args : gst_plugins_base_args + [sse_args], + c_args : gst_plugins_base_args + [sse_args] + [pic_args], include_directories : [configinc, libsinc], dependencies : [gst_base_dep], - install : false, - pic: true + install : false ) simd_cargs += ['-DHAVE_SSE'] @@ -105,11 +104,10 @@ endif if have_sse2 audio_resampler_sse2 = static_library('audio_resampler_sse2', ['audio-resampler-x86-sse2.c', gstaudio_h], - c_args : gst_plugins_base_args + [sse2_args], + c_args : gst_plugins_base_args + [sse2_args] + [pic_args], include_directories : [configinc, libsinc], dependencies : [gst_base_dep], - install : false, - pic: true + install : false ) simd_cargs += ['-DHAVE_SSE2'] @@ -119,11 +117,10 @@ endif if have_sse41 audio_resampler_sse41 = static_library('audio_resampler_sse41', ['audio-resampler-x86-sse41.c', gstaudio_h], - c_args : gst_plugins_base_args + [sse41_args], + c_args : gst_plugins_base_args + [sse41_args] + [pic_args], include_directories : [configinc, libsinc], dependencies : [gst_base_dep], - install : false, - pic: true + install : false ) simd_cargs += ['-DHAVE_SSE41'] diff --git a/meson.build b/meson.build index ec4ce96309..a0eaf084d6 100644 --- a/meson.build +++ b/meson.build @@ -185,6 +185,14 @@ have_sse = cc.has_argument(sse_args) have_sse2 = cc.has_argument(sse2_args) have_sse41 = cc.has_argument(sse41_args) +# FIXME: Meson should have a way for portably adding -fPIC when needed for use +# with static libraries that are linked into shared libraries. Or, it should +# add it by default with an option to turn it off if needed. +pic_args = ['-fPIC'] +if host_machine.system() == 'windows' + pic_args = [] +endif + subdir('gst-libs') subdir('gst') subdir('ext')