commit: 7855f572f7b201eebe1476a9dc01ec2726f8ce88 Author: Mathy Vanvoorden <mathy <AT> vanvoorden <DOT> be> AuthorDate: Thu Jun 14 21:15:04 2018 +0000 Commit: Alexey Shvetsov <alexxy <AT> gentoo <DOT> org> CommitDate: Fri Jun 15 11:09:08 2018 +0000 URL: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=7855f572
dev-libs/libsavitar: use pugixml from system resolving file collision Upstream put pugixml inside the distribution and it is built and installed alongside with the library. This results in file collisions when pugixml is already installed on the system. Existing upstream pull request: https://github.com/Ultimaker/libSavitar/pull/8 This commit applies that pull request as a patch. I modified the patch slightly to not include the actual deletion of the library and only apply the needed changes to the rest of the library. I did this because otherwise the filesize was bigger than the portage limit. Closes: https://bugs.gentoo.org/658112 Closes: https://github.com/gentoo/gentoo/pull/8846 Package-Manager: Portage[mgorny]-2.3.36.1 .../libsavitar-3.3.0-remove-packaged-pugixml.patch | 119 +++++++++++++++++++++ ...tar-3.3.0.ebuild => libsavitar-3.3.0-r1.ebuild} | 2 + 2 files changed, 121 insertions(+) diff --git a/dev-libs/libsavitar/files/libsavitar-3.3.0-remove-packaged-pugixml.patch b/dev-libs/libsavitar/files/libsavitar-3.3.0-remove-packaged-pugixml.patch new file mode 100644 index 00000000000..629e8edb51a --- /dev/null +++ b/dev-libs/libsavitar/files/libsavitar-3.3.0-remove-packaged-pugixml.patch @@ -0,0 +1,119 @@ +From 91aa70c6320461ae65600dce954b5ffb905b7039 Mon Sep 17 00:00:00 2001 +From: Eli Schwartz <eschwart...@gmail.com> +Date: Mon, 9 Apr 2018 12:36:35 -0400 +Subject: [PATCH] Do not vendor the pugixml library + +This prevents trying to co-install pugixml with existing system +libraries, and also fixes the includes for pugixml.hpp to actually use +the include path added by find_package/add_subdirectory instead of a +hardcoded relative path. +--- + CMakeLists.txt | 3 +- + pugixml/CMakeLists.txt | 80 - + pugixml/src/pugiconfig.hpp | 74 - + pugixml/src/pugixml.cpp | 12622 ------------------------------------------- + pugixml/src/pugixml.hpp | 1434 ----- + src/MeshData.cpp | 7 +- + src/Scene.cpp | 2 +- + src/SceneNode.cpp | 3 +- + src/ThreeMFParser.h | 4 +- + 9 files changed, 7 insertions(+), 14222 deletions(-) + delete mode 100644 pugixml/CMakeLists.txt + delete mode 100644 pugixml/src/pugiconfig.hpp + delete mode 100644 pugixml/src/pugixml.cpp + delete mode 100644 pugixml/src/pugixml.hpp + +diff --git a/CMakeLists.txt b/CMakeLists.txt +index 5e991a7..1077c7f 100644 +--- a/CMakeLists.txt ++++ b/CMakeLists.txt +@@ -7,8 +7,7 @@ include(CMakePackageConfigHelpers) + option(BUILD_PYTHON "Build " ON) + option(BUILD_STATIC "Build as a static library" OFF) + +- +-add_subdirectory(pugixml) ++find_package(pugixml REQUIRED) + + if(BUILD_PYTHON) + set(CMAKE_MODULE_PATH ${CMAKE_SOURCE_DIR}/cmake) +diff --git a/src/MeshData.cpp b/src/MeshData.cpp +index 03e5089..701122d 100644 +--- a/src/MeshData.cpp ++++ b/src/MeshData.cpp +@@ -17,7 +17,7 @@ + */ + + #include "MeshData.h" +-#include "../pugixml/src/pugixml.hpp" ++#include <pugixml.hpp> + #include <iostream> + + using namespace Savitar; +@@ -85,7 +85,7 @@ bytearray MeshData::getFlatVerticesAsBytes() + int v1 = faces.at(i).getV1(); + int v2 = faces.at(i).getV2(); + int v3 = faces.at(i).getV3(); +- ++ + // Add vertices for face 1 + float x = vertices.at(v1).getX(); + float y = vertices.at(v1).getY(); +@@ -188,6 +188,3 @@ std::vector< Vertex > MeshData::getVertices() + { + return vertices; + } +- +- +- +diff --git a/src/Scene.cpp b/src/Scene.cpp +index 4612f78..f218b71 100644 +--- a/src/Scene.cpp ++++ b/src/Scene.cpp +@@ -17,7 +17,7 @@ + */ + + #include "Scene.h" +-#include "../pugixml/src/pugixml.hpp" ++#include <pugixml.hpp> + #include <iostream> + #include <string> + using namespace Savitar; +diff --git a/src/SceneNode.cpp b/src/SceneNode.cpp +index 5c95b6f..c4782bc 100644 +--- a/src/SceneNode.cpp ++++ b/src/SceneNode.cpp +@@ -17,7 +17,7 @@ + */ + + #include "SceneNode.h" +-#include "../pugixml/src/pugixml.hpp" ++#include <pugixml.hpp> + #include <iostream> + using namespace Savitar; + +@@ -121,4 +121,3 @@ std::vector< SceneNode*> SceneNode::getAllChildren() + } + return all_children; + } +- +diff --git a/src/ThreeMFParser.h b/src/ThreeMFParser.h +index 3394683..89da659 100644 +--- a/src/ThreeMFParser.h ++++ b/src/ThreeMFParser.h +@@ -21,7 +21,7 @@ + + #include "SavitarExport.h" + #include "SceneNode.h" +-#include "../pugixml/src/pugixml.hpp" ++#include <pugixml.hpp> + + #include <string> + namespace Savitar +@@ -44,4 +44,4 @@ namespace Savitar + std::string sceneToString(Scene scene); + }; + } +-#endif +\ No newline at end of file ++#endif diff --git a/dev-libs/libsavitar/libsavitar-3.3.0.ebuild b/dev-libs/libsavitar/libsavitar-3.3.0-r1.ebuild similarity index 93% rename from dev-libs/libsavitar/libsavitar-3.3.0.ebuild rename to dev-libs/libsavitar/libsavitar-3.3.0-r1.ebuild index 7a46bfd8a4a..3aca6609e16 100644 --- a/dev-libs/libsavitar/libsavitar-3.3.0.ebuild +++ b/dev-libs/libsavitar/libsavitar-3.3.0-r1.ebuild @@ -25,6 +25,8 @@ DEPEND="${RDEPEND}" S="${WORKDIR}/${MY_PN}-${PV}" +PATCHES=( "${FILESDIR}"/${P}-remove-packaged-pugixml.patch ) + pkg_setup() { use python && python-single-r1_pkg_setup }
