remove non-free xmlParser.cpp

.gitignore

@@ -1 +1,2 @@
 /libmusicbrainz-5.0.1.tar.gz
+/libmusicbrainz-5.0.1-clean.tar.gz

libmusicbrainz5.spec

@@ -4,15 +4,22 @@
 Summary: Library for accessing MusicBrainz servers
 Name: libmusicbrainz5
 Version: 5.0.1
-Release: 2%{?dist}
+Release: 3%{?dist}
 License: LGPLv2
 Group: System Environment/Libraries
 URL: http://www.musicbrainz.org/
-Source0: https://github.com/downloads/metabrainz/libmusicbrainz/libmusicbrainz-%{version}.tar.gz
+# Source0: https://github.com/downloads/metabrainz/libmusicbrainz/libmusicbrainz-%{version}.tar.gz
-
+# We need to remove src/xmlParser.cpp, as it is non-free and prohibits commercial distribution.
+# Just unpack and rm -rf src/xmlParser.cpp, then 
+# tar cfz libmusicbrainz-%{version}-clean.tar.gz libmusicbrainz-%{version}/
+Source0: libmusicbrainz-%{version}-clean.tar.gz
+# This patch replaces the removed src/xmlParser.cpp support with libxml2
+# and fixes up the cinterface.xml file (libxml2 seems to be stricter).
+Patch0: 0001-xmlparsing.patch
 BuildRequires: cmake
 BuildRequires: doxygen
 BuildRequires: neon-devel
+BuildRequires: libxml2-devel
 
 %description
 The MusicBrainz client library allows applications to make metadata
@@ -30,6 +37,7 @@ applications which will use %{name}.
 
 %prep
 %setup -q -n libmusicbrainz-%{version}
+%patch0 -p1 -b .xmlParser
 
 # omit "Generated on ..." timestamps that induce multilib conflicts
 # this is *supposed* to be the doxygen default in fedora these days, but
@@ -68,6 +76,9 @@ rm -f docs/installdox
 
 
 %changelog
+* Mon Jan  7 2013 Tom Callaway <spot@fedoraproject.org> - 5.0.1-3
+- remove non-free xmlParser.cpp
+
 * Mon Aug 27 2012 Christophe Fergeau <cfergeau@redhat.com> - 5.0.1-2
 - Address review comments (rhbz#832670)
     fix licence (LGPLv2+ -> LGPLv2)

sources

@@ -1 +1 @@
-a0406b94c341c2b52ec0fe98f57cadf3  libmusicbrainz-5.0.1.tar.gz
+1abebd606bca260c5f5f6936f6e304ff  libmusicbrainz-5.0.1-clean.tar.gz