Hello community,

here is the log from the commit of package libmatroska for openSUSE:Factory 
checked in at 2014-01-20 20:42:39
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/libmatroska (Old)
 and      /work/SRC/openSUSE:Factory/.libmatroska.new (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "libmatroska"

Changes:
--------
--- /work/SRC/openSUSE:Factory/libmatroska/libmatroska.changes  2013-11-04 
07:07:07.000000000 +0100
+++ /work/SRC/openSUSE:Factory/.libmatroska.new/libmatroska.changes     
2014-01-23 15:47:54.000000000 +0100
@@ -1,0 +2,15 @@
+Thu Jan 16 17:58:40 UTC 2014 - [email protected]
+
+- Revert last change for now as it breaks mkvtoolnix which 
+  uses internal symbols :-(
+  keep fvisibility-inlines-hidden though
+
+-------------------------------------------------------------------
+Sun Jan 12 02:26:29 UTC 2014 - [email protected]
+
+- Add C++ visibility support by simply reusing the win32 
+  dll code for the same purpose.
+- added patches:
+  * matroska-visibility.patch
+
+-------------------------------------------------------------------

New:
----
  matroska-visibility.patch

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ libmatroska.spec ++++++
--- /var/tmp/diff_new_pack.JwbK5b/_old  2014-01-23 15:47:55.000000000 +0100
+++ /var/tmp/diff_new_pack.JwbK5b/_new  2014-01-23 15:47:55.000000000 +0100
@@ -1,7 +1,7 @@
 #
 # spec file for package libmatroska
 #
-# Copyright (c) 2013 SUSE LINUX Products GmbH, Nuernberg, Germany.
+# Copyright (c) 2014 SUSE LINUX Products GmbH, Nuernberg, Germany.
 #
 # All modifications and additions to the file contributed by third parties
 # remain the property of their copyright owners, unless otherwise agreed
@@ -29,6 +29,7 @@
 Source1:        baselibs.conf
 BuildRequires:  gcc-c++
 BuildRequires:  libebml-devel >= 1.3.0
+Patch0:         matroska-visibility.patch
 
 %description
 Libmatroska is a C++ library to parse Matroska files (.mkv and .mka).
@@ -59,10 +60,11 @@
 %setup -q
 FAKE_BUILDTIMESTAMP=$(LC_ALL=C date -r %{_sourcedir}/%{name}.changes '+%a %b 
%e %H:%M:%S %Y')
 sed -i "s/__TIMESTAMP__/\"$FAKE_BUILDTIMESTAMP\"/" matroska/KaxVersion.h
+#%patch0 -p1
 
 %build
 cd make/linux
-export CXXFLAGS='%{optflags}'
+export CXXFLAGS='%{optflags} -fvisibility-inlines-hidden'
 make %{?_smp_mflags} link=shared
 
 %install

++++++ matroska-visibility.patch ++++++
--- libmatroska-1.4.1.orig/make/linux/Makefile
+++ libmatroska-1.4.1/make/linux/Makefile
@@ -77,7 +77,7 @@ objects:=$(patsubst %$(EXTENSION),%.o,$(
 
 objects_so:=$(patsubst %$(EXTENSION),%.lo,$(sources))
 
-WARNINGFLAGS=-Wall -Wextra -Wno-unknown-pragmas -D_GNU_SOURCE -Wshadow
+WARNINGFLAGS=-Wall -Wextra -Wno-unknown-pragmas -D_GNU_SOURCE -Wshadow 
-fvisibility=hidden -fvisibility-inlines-hidden
 COMPILEFLAGS=$(DEBUGFLAGS) $(CXXFLAGS) $(CPPFLAGS) $(WARNINGFLAGS) $(INCLUDE)
 LINKFLAGS=-L. -L$(LIBEBML_LIB_DIR) $(LDFLAGS)
 DEPENDFLAGS  = $(CXXFLAGS) $(INCLUDE)
@@ -104,7 +104,7 @@ $(LIBRARY): $(objects)
        $(RANLIB) $@
 
 $(LIBRARY_SO): $(objects_so)
-       $(CXX) -shared -Wl,-soname,$(LIBRARY_SO_VER) -o $(LIBRARY_SO_VER) 
$(objects_so) -lebml
+       $(CXX) $(COMPILEFLAGS) -fPIC -Wl,--no-undefined -shared 
-Wl,-soname,$(LIBRARY_SO_VER) -o $(LIBRARY_SO_VER) $(objects_so) -lebml
        rm -f $(LIBRARY_SO)
        ln -s $(LIBRARY_SO_VER) $(LIBRARY_SO)
 
--- libmatroska-1.4.1.orig/matroska/KaxConfig.h
+++ libmatroska-1.4.1/matroska/KaxConfig.h
@@ -59,7 +59,7 @@
 # endif // MATROSKA_DLL
 
 #else
-# define MATROSKA_DLL_API
+# define MATROSKA_DLL_API __attribute__ ((visibility ("default")))
 #endif
 
 #if !defined(MATROSKA_VERSION)
-- 
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to