[vlc-devel] [vlc-commits] contrib/ebml+mkv: compile with hidden symbols

[Steve Lhomme]

What was the error when this wasn't set ? I'd like to fix this in the
original libraries so the patch is not needed anymore.

On Thu, Apr 30, 2015 at 8:34 PM, Felix Paul Kühne <git at videolan.org> wrote:
> vlc/vlc-2.2 | branch: master | Felix Paul Kühne <fkuehne at videolan.org> | Sat Jan 24 23:53:59 2015 +0100| [5c5cd0c3d6ab5aedc909ea0d2ebde3d74ed7cff4] | committer: Felix Paul Kühne
>
> contrib/ebml+mkv: compile with hidden symbols
>
> This solves linking issues on iOS and should have no negative impact on further platforms
>
> (cherry picked from commit 6031c9769a0dbd8a4f484027d5856b12ec9cc188)
>
>> http://git.videolan.org/gitweb.cgi/vlc/vlc-2.2.git/?a=commit;h=5c5cd0c3d6ab5aedc909ea0d2ebde3d74ed7cff4
> ---
>
>  contrib/src/ebml/rules.mak     |    4 ++--
>  contrib/src/matroska/rules.mak |    4 +++-
>  2 files changed, 5 insertions(+), 3 deletions(-)
>
> diff --git a/contrib/src/ebml/rules.mak b/contrib/src/ebml/rules.mak
> index 9d87c29..4a5ac16 100644
> --- a/contrib/src/ebml/rules.mak
> +++ b/contrib/src/ebml/rules.mak
> @@ -14,10 +14,10 @@ libebml: libebml-$(EBML_VERSION).tar.bz2 .sum-ebml
>         $(MOVE)
>
>  # libebml requires exceptions
> -EBML_EXTRA_FLAGS = CXXFLAGS="${CXXFLAGS} -fexceptions" \
> +EBML_EXTRA_FLAGS = CXXFLAGS="${CXXFLAGS} -fexceptions -fvisibility=hidden" \
>                                         CPPFLAGS=""
>
>  .ebml: libebml
> -       cd $< && $(HOSTVARS) ./configure $(HOSTCONF)
> +       cd $< && $(HOSTVARS) ./configure $(HOSTCONF) $(EBML_EXTRA_FLAGS)
>         cd $< && $(MAKE) install
>         touch $@
> diff --git a/contrib/src/matroska/rules.mak b/contrib/src/matroska/rules.mak
> index b8d0165..c71ee83 100644
> --- a/contrib/src/matroska/rules.mak
> +++ b/contrib/src/matroska/rules.mak
> @@ -16,7 +16,9 @@ libmatroska: libmatroska-$(MATROSKA_VERSION).tar.bz2 .sum-matroska
>         $(UNPACK)
>         $(MOVE)
>
> +MATROSKA_EXTRA_FLAGS = CXXFLAGS="${CXXFLAGS} -fvisibility=hidden"
> +
>  .matroska: libmatroska
> -       cd $< && $(HOSTVARS) ./configure $(HOSTCONF)
> +       cd $< && $(HOSTVARS) ./configure $(HOSTCONF) $(MATROSKA_EXTRA_FLAGS)
>         cd $< && $(MAKE) install
>         touch $@
>
> _______________________________________________
> vlc-commits mailing list
> vlc-commits at videolan.org
> https://mailman.videolan.org/listinfo/vlc-commits