public inbox for gentoo-commits@lists.gentoo.org
 help / color / mirror / Atom feed
From: "David Seifert" <soap@gentoo.org>
To: gentoo-commits@lists.gentoo.org
Subject: [gentoo-commits] repo/gentoo:master commit in: media-gfx/openvdb/, media-gfx/openvdb/files/
Date: Thu, 27 Oct 2016 18:34:28 +0000 (UTC)	[thread overview]
Message-ID: <1477593251.f0504839a891e2a6fa94dd3da3e520c8b9dc8cb3.soap@gentoo> (raw)

commit:     f0504839a891e2a6fa94dd3da3e520c8b9dc8cb3
Author:     Jonathan Scruggs <j.scruggs <AT> gmail <DOT> com>
AuthorDate: Thu Oct 27 12:16:22 2016 +0000
Commit:     David Seifert <soap <AT> gentoo <DOT> org>
CommitDate: Thu Oct 27 18:34:11 2016 +0000
URL:        https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=f0504839

media-gfx/openvdb: replace -isystem with -I for GCC 6 compatibility

Gentoo-bug: 597866
* Respect CXX
* Respect ABI libdirs
* Remove '-O3' from compile lines

Signed-off by: Jonathan Scruggs (j.scruggs <AT> gmail.com)
Closes: https://github.com/gentoo/gentoo/pull/2677

Signed-off-by: David Seifert <soap <AT> gentoo.org>

 .../openvdb/files/openvdb-3.2.0-build-fixes.patch  |  33 ---
 .../files/openvdb-3.2.0-fix-build-system.patch     | 262 +++++++++++++++++++++
 .../files/openvdb-3.2.0-makefile-fixes.patch       |  93 --------
 media-gfx/openvdb/openvdb-3.2.0.ebuild             |  24 +-
 4 files changed, 276 insertions(+), 136 deletions(-)

diff --git a/media-gfx/openvdb/files/openvdb-3.2.0-build-fixes.patch b/media-gfx/openvdb/files/openvdb-3.2.0-build-fixes.patch
deleted file mode 100644
index e919ec2..00000000
--- a/media-gfx/openvdb/files/openvdb-3.2.0-build-fixes.patch
+++ /dev/null
@@ -1,33 +0,0 @@
-diff -purN a/doxygen-config b/doxygen-config
---- a/doxygen-config	2016-08-10 16:59:33.000000000 +0100
-+++ b/doxygen-config	2016-10-12 12:19:06.124785164 +0100
-@@ -2257,7 +2257,7 @@ DIRECTORY_GRAPH        = YES
- # The default value is: png.
- # This tag requires that the tag HAVE_DOT is set to YES.
- 
--DOT_IMAGE_FORMAT       = png
-+DOT_IMAGE_FORMAT       = svg
- 
- # If DOT_IMAGE_FORMAT is set to svg, then this option can be set to YES to
- # enable generation of interactive SVG images that allow zooming and panning.
-diff -purN a/Makefile b/Makefile
---- a/Makefile	2016-08-10 16:59:33.000000000 +0100
-+++ b/Makefile	2016-10-12 12:18:32.434773635 +0100
-@@ -678,7 +678,7 @@ ALL_PRODUCTS := \
- 	@echo "Building $@ because of $(call list_deps)"
- 	$(CXX) -c $(CXXFLAGS) -fPIC -o $@ $<
- 
--all: lib python vdb_print vdb_render vdb_test depend
-+all: lib python vdb_print vdb_render vdb_view vdb_test depend
- 
- $(OBJ_NAMES): %.o: %.cc
- 	@echo "Building $@ because of $(call list_deps)"
-@@ -794,7 +794,7 @@ pydoc: $(PYTHON_MODULE) $(LIBOPENVDB_SON
- 	echo "Created $${pydocdir}"; \
- 	export LD_LIBRARY_PATH=${LD_LIBRARY_PATH}:$(CURDIR); \
- 	export PYTHONPATH=${PYTHONPATH}:$(CURDIR); \
--	    $(EPYDOC) --html -o $${pydocdir} $(PYTHON_MODULE_NAME) $(QUIET)
-+	    $(EPYDOC) --html --html-dir $${pydocdir} $(PYTHON_MODULE_NAME) $(QUIET)
- else
- pydoc:
- 	@echo "$@"': $$EPYDOC is undefined'

diff --git a/media-gfx/openvdb/files/openvdb-3.2.0-fix-build-system.patch b/media-gfx/openvdb/files/openvdb-3.2.0-fix-build-system.patch
new file mode 100644
index 00000000..66112af
--- /dev/null
+++ b/media-gfx/openvdb/files/openvdb-3.2.0-fix-build-system.patch
@@ -0,0 +1,262 @@
+* Make build system honour the proper ABI library directory.
+* Replace -isystem arguments with -I, as the former causes issues with GCC 6.
+  See also:
+  https://bugs.gentoo.org/show_bug.cgi?id=597866
+  https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70129
+
+--- a/doxygen-config
++++ b/doxygen-config
+@@ -2257,7 +2257,7 @@
+ # The default value is: png.
+ # This tag requires that the tag HAVE_DOT is set to YES.
+ 
+-DOT_IMAGE_FORMAT       = png
++DOT_IMAGE_FORMAT       = svg
+ 
+ # If DOT_IMAGE_FORMAT is set to svg, then this option can be set to YES to
+ # enable generation of interactive SVG images that allow zooming and panning.
+--- a/Makefile
++++ b/Makefile
+@@ -82,6 +82,7 @@
+ 
+ # The directory into which to install libraries, executables and header files
+ DESTDIR := /tmp/OpenVDB
++libdir = lib
+ 
+ # The parent directory of the boost/ header directory
+ BOOST_INCL_DIR := $(HT)/include
+@@ -158,6 +159,9 @@
+ # The directory containing libpython
+ PYTHON_LIB_DIR := $(HFS)/python/lib
+ PYTHON_LIB := -lpython$(PYTHON_VERSION)
++# The Directory to install the python module and includes to.
++PYTHON_INSTALL_INCL_DIR := $(DESTDIR)/python/include/python$(PYTHON_VERSION)
++PYTHON_INSTALL_LIB_DIR := $(DESTDIR)/python/lib/python$(PYTHON_VERSION)
+ # The directory containing libboost_python
+ BOOST_PYTHON_LIB_DIR := /rel/depot/third_party_build/boost/rhel6-1.46.1-0/lib
+ BOOST_PYTHON_LIB := -lboost_python-gcc41-mt-python26-1_46_1
+@@ -179,7 +183,6 @@
+ # (leave blank if Doxygen is unavailable)
+ DOXYGEN := doxygen
+ 
+-
+ #
+ # Ideally, users shouldn't need to change anything below this line.
+ #
+@@ -207,11 +210,7 @@
+     endif
+ endif
+ 
+-ifeq (yes,$(strip $(debug)))
+-    OPTIMIZE := -g
+-else
+-    OPTIMIZE := -O3 -DNDEBUG
+-endif
++OPTIMIZE := -DNDEBUG
+ 
+ ifeq (yes,$(strip $(verbose)))
+     QUIET :=
+@@ -242,12 +241,12 @@
+     has_python := yes
+ endif
+ 
+-INCLDIRS := -I . -I .. -isystem $(BOOST_INCL_DIR) -isystem $(ILMBASE_INCL_DIR) -isystem $(TBB_INCL_DIR)
++INCLDIRS := -I. -I.. -I$(BOOST_INCL_DIR) -I$(ILMBASE_INCL_DIR) -I$(TBB_INCL_DIR)
+ ifeq (yes,$(has_blosc))
+-    INCLDIRS += -isystem $(BLOSC_INCL_DIR)
++    INCLDIRS += -I$(BLOSC_INCL_DIR)
+ endif
+ ifeq (yes,$(has_log4cplus))
+-    INCLDIRS += -isystem $(LOG4CPLUS_INCL_DIR)
++    INCLDIRS += -I$(LOG4CPLUS_INCL_DIR)
+ endif
+ 
+ CXXFLAGS += -std=c++0x
+@@ -565,9 +564,9 @@
+     python/pyTransform.cc \
+     python/pyVec3Grid.cc \
+ #
+-PYCXXFLAGS := -fPIC -isystem python -isystem $(PYTHON_INCL_DIR) -isystem $(PYCONFIG_INCL_DIR)
++PYCXXFLAGS := -fPIC -Ipython -I$(PYTHON_INCL_DIR) -I$(PYCONFIG_INCL_DIR)
+ ifneq (,$(strip $(NUMPY_INCL_DIR)))
+-PYCXXFLAGS += -isystem $(NUMPY_INCL_DIR) -DPY_OPENVDB_USE_NUMPY
++PYCXXFLAGS += -I$(NUMPY_INCL_DIR) -DPY_OPENVDB_USE_NUMPY
+ endif
+ ifneq (no,$(strip $(PYTHON_WRAP_ALL_GRID_TYPES)))
+ PYCXXFLAGS += -DPY_OPENVDB_WRAP_ALL_GRID_TYPES
+@@ -678,7 +677,7 @@
+ 	@echo "Building $@ because of $(call list_deps)"
+ 	$(CXX) -c $(CXXFLAGS) -fPIC -o $@ $<
+ 
+-all: lib python vdb_print vdb_render vdb_test depend
++all: lib python vdb_print vdb_render vdb_view vdb_test depend
+ 
+ $(OBJ_NAMES): %.o: %.cc
+ 	@echo "Building $@ because of $(call list_deps)"
+@@ -719,8 +718,13 @@
+ 	@echo "Generating documentation because of $(list_deps)"
+ 	echo -e 'OUTPUT_DIRECTORY=./doc\nGENERATE_LATEX=YES\nGENERATE_HTML=NO' \
+ 	    | cat doxygen-config - | $(DOXYGEN) - $(QUIET) \
+-	    && cd ./doc/latex && make refman.pdf $(QUIET) \
++	    && cd ./doc/latex && $(MAKE) refman.pdf $(QUIET) \
+ 	    && echo 'Created doc/latex/refman.pdf'
++	@#
++	    mkdir -p $(DESTDIR)/share/doc/openvdb/pdf; \
++	    echo "Created $(DESTDIR)/share/doc/openvdb/pdf"; \
++	    cp -r -f doc/latex/refman.pdf $(DESTDIR)/share/doc/openvdb/pdf/; \
++	    echo "Copied documentation to $(DESTDIR)/share/doc/openvdb/pdf/"; \
+ 
+ ifneq (,$(strip $(DOXYGEN)))
+ doc: $(DOC_INDEX)
+@@ -734,14 +738,14 @@
+ 
+ vdb_print: $(LIBOPENVDB) cmd/openvdb_print/main.cc
+ 	@echo "Building $@ because of $(list_deps)"
+-	$(CXX) $(CXXFLAGS) -o $@ cmd/openvdb_print/main.cc -I . \
++	$(CXX) $(CXXFLAGS) -o $@ cmd/openvdb_print/main.cc -I. \
+ 	    $(LIBOPENVDB_RPATH) -L$(CURDIR) $(LIBOPENVDB) \
+ 	    $(LIBS_RPATH) $(CONCURRENT_MALLOC_LIB)
+ 
+ vdb_render: $(LIBOPENVDB) cmd/openvdb_render/main.cc
+ 	@echo "Building $@ because of $(list_deps)"
+-	$(CXX) $(CXXFLAGS) -o $@ cmd/openvdb_render/main.cc -I . \
+-	    -isystem $(EXR_INCL_DIR) -isystem $(ILMBASE_INCL_DIR) \
++	$(CXX) $(CXXFLAGS) -o $@ cmd/openvdb_render/main.cc -I. \
++	    -I$(EXR_INCL_DIR) -I$(ILMBASE_INCL_DIR) \
+ 	    -Wl,-rpath,$(EXR_LIB_DIR) -L$(EXR_LIB_DIR) $(EXR_LIB) \
+ 	    -Wl,-rpath,$(ILMBASE_LIB_DIR) -L$(ILMBASE_LIB_DIR) $(ILMBASE_LIB) \
+ 	    $(LIBOPENVDB_RPATH) -L$(CURDIR) $(LIBOPENVDB) \
+@@ -761,12 +765,12 @@
+ $(LIBVIEWER_OBJ_NAMES): $(LIBVIEWER_INCLUDE_NAMES)
+ $(LIBVIEWER_OBJ_NAMES): %.o: %.cc
+ 	@echo "Building $@ because of $(list_deps)"
+-	$(CXX) -c $(CXXFLAGS) -I . -isystem $(GLFW_INCL_DIR) -DGL_GLEXT_PROTOTYPES=1 -fPIC -o $@ $<
++	$(CXX) -c $(CXXFLAGS) -I. -I$(GLFW_INCL_DIR) -DGL_GLEXT_PROTOTYPES=1 -fPIC -o $@ $<
+ 
+ vdb_view: $(LIBOPENVDB) $(LIBVIEWER_OBJ_NAMES) cmd/openvdb_view/main.cc
+ 	@echo "Building $@ because of $(list_deps)"
+ 	$(CXX) $(CXXFLAGS) -o $@ cmd/openvdb_view/main.cc $(LIBVIEWER_OBJ_NAMES) \
+-	    -I . -Wl,-rpath,$(GLFW_LIB_DIR) -L$(GLFW_LIB_DIR) $(GLFW_LIB) \
++	    -I. -Wl,-rpath,$(GLFW_LIB_DIR) -L$(GLFW_LIB_DIR) $(GLFW_LIB) \
+ 	    $(LIBOPENVDB_RPATH) -L$(CURDIR) $(LIBOPENVDB) \
+ 	    $(LIBVIEWER_FLAGS) $(LIBS_RPATH) $(BOOST_THREAD_LIB) $(CONCURRENT_MALLOC_LIB)
+ endif
+@@ -776,7 +780,7 @@
+ $(PYTHON_OBJ_NAMES): $(PYTHON_INCLUDE_NAMES)
+ $(PYTHON_OBJ_NAMES): %.o: %.cc
+ 	@echo "Building $@ because of $(list_deps)"
+-	$(CXX) -c $(CXXFLAGS) -I . $(PYCXXFLAGS) -o $@ $<
++	$(CXX) -c $(CXXFLAGS) -I. $(PYCXXFLAGS) -o $@ $<
+ $(PYTHON_MODULE): $(LIBOPENVDB) $(PYTHON_OBJ_NAMES)
+ 	@echo "Building $@ because of $(list_deps)"
+ 	$(CXX) $(CXXFLAGS) $(PYCXXFLAGS) -shared $(PYTHON_SONAME_FLAGS) -o $@ $(PYTHON_OBJ_NAMES) \
+@@ -784,6 +788,22 @@
+ 	    -Wl,-rpath,$(BOOST_PYTHON_LIB_DIR) -L$(BOOST_PYTHON_LIB_DIR) $(BOOST_PYTHON_LIB) \
+ 	    $(LIBOPENVDB_RPATH) -L$(CURDIR) $(LIBOPENVDB) \
+ 	    $(LIBS_RPATH) $(CONCURRENT_MALLOC_LIB)
++	@echo "Installing the python module and includes"
++	@#
++	    mkdir -p $(PYTHON_INSTALL_INCL_DIR); \
++	    echo "Created $(PYTHON_INSTALL_INCL_DIR)"; \
++	    cp -f $(PYTHON_PUBLIC_INCLUDE_NAMES) $(PYTHON_INSTALL_INCL_DIR)/; \
++	    echo "Copied Python header files to $(PYTHON_INSTALL_INCL_DIR)"; \
++	    mkdir -p $(PYTHON_INSTALL_LIB_DIR); \
++	    echo "Created $(PYTHON_INSTALL_LIB_DIR)"; \
++	    cp -f $(PYTHON_MODULE) $(PYTHON_INSTALL_LIB_DIR)/; \
++	    pushd $(PYTHON_INSTALL_LIB_DIR) > /dev/null; \
++	    ln -f -s $(PYTHON_MODULE) $(PYTHON_SONAME); \
++	    popd > /dev/null; \
++	    echo "Copied Python module to $(PYTHON_INSTALL_LIB_DIR)"
++	@echo "Clean the Python modules so the next one will compile"
++	@#
++	    $(RM) $(PYTHON_OBJ_NAMES)
+ 
+ ifeq (yes,$(has_python))
+ ifneq (,$(strip $(EPYDOC)))
+@@ -794,7 +814,7 @@
+ 	echo "Created $${pydocdir}"; \
+ 	export LD_LIBRARY_PATH=${LD_LIBRARY_PATH}:$(CURDIR); \
+ 	export PYTHONPATH=${PYTHONPATH}:$(CURDIR); \
+-	    $(EPYDOC) --html -o $${pydocdir} $(PYTHON_MODULE_NAME) $(QUIET)
++	    $(EPYDOC) --html --html-dir $${pydocdir} $(PYTHON_MODULE_NAME) $(QUIET)
+ else
+ pydoc:
+ 	@echo "$@"': $$EPYDOC is undefined'
+@@ -815,7 +835,7 @@
+ 
+ $(UNITTEST_OBJ_NAMES): %.o: %.cc
+ 	@echo "Building $@ because of $(list_deps)"
+-	$(CXX) -c $(CXXFLAGS) -isystem $(CPPUNIT_INCL_DIR) -fPIC -o $@ $<
++	$(CXX) -c $(CXXFLAGS) -I$(CPPUNIT_INCL_DIR) -fPIC -o $@ $<
+ 
+ ifneq (,$(strip $(CPPUNIT_INCL_DIR)))
+ vdb_test: $(LIBOPENVDB) $(UNITTEST_OBJ_NAMES)
+@@ -835,7 +855,7 @@
+ 	@echo "$@"': $$(CPPUNIT_INCL_DIR) is undefined'
+ endif
+ 
+-install: lib python vdb_print vdb_render vdb_view doc pydoc
++install: lib vdb_print vdb_render vdb_view doc pydoc
+ 	mkdir -p $(DESTDIR)/include/openvdb
+ 	@echo "Created $(DESTDIR)/include/openvdb"
+ 	pushd $(DESTDIR)/include/openvdb > /dev/null; \
+@@ -851,42 +871,25 @@
+ 	fi
+ 	@echo "Copied header files to $(DESTDIR)/include"
+ 	@#
+-	mkdir -p $(DESTDIR)/lib
+-	@echo "Created $(DESTDIR)/lib/"
+-	cp -f $(LIBOPENVDB) $(DESTDIR)/lib
+-	pushd $(DESTDIR)/lib > /dev/null; \
++	mkdir -p $(DESTDIR)/$(libdir)
++	@echo "Created $(DESTDIR)/$(libdir)/"
++	cp -f $(LIBOPENVDB) $(DESTDIR)/$(libdir)
++	pushd $(DESTDIR)/$(libdir) > /dev/null; \
+ 	    if [ -f $(LIBOPENVDB_SHARED) ]; then \
+ 	        ln -f -s $(LIBOPENVDB_SHARED) $(LIBOPENVDB_SHARED_NAME); \
+ 	        ln -f -s $(LIBOPENVDB_SHARED) $(LIBOPENVDB_SONAME); \
+ 	    fi; \
+ 	    popd > /dev/null
+-	@echo "Copied libopenvdb to $(DESTDIR)/lib/"
++	@echo "Copied libopenvdb to $(DESTDIR)/$(libdir)/"
+ 	@#
+ 	if [ -f $(LIBVIEWER) ]; \
+ 	then \
+-	    cp -f $(LIBVIEWER) $(DESTDIR)/lib; \
+-	    pushd $(DESTDIR)/lib > /dev/null; \
++	    cp -f $(LIBVIEWER) $(DESTDIR)/$(libdir); \
++	    pushd $(DESTDIR)/$(libdir) > /dev/null; \
+ 	        if [ -f $(LIBVIEWER_SHARED) ]; then \
+ 	            ln -f -s $(LIBVIEWER_SHARED) $(LIBVIEWER_SHARED_NAME); fi; \
+ 	        popd > /dev/null; \
+-	    echo "Copied libopenvdb_viewer to $(DESTDIR)/lib/"; \
+-	fi
+-	@#
+-	if [ -f $(PYTHON_MODULE) ]; \
+-	then \
+-	    installdir=$(DESTDIR)/python/include/python$(PYTHON_VERSION); \
+-	    mkdir -p $${installdir}; \
+-	    echo "Created $${installdir}"; \
+-	    cp -f $(PYTHON_PUBLIC_INCLUDE_NAMES) $${installdir}/; \
+-	    echo "Copied Python header files to $${installdir}"; \
+-	    installdir=$(DESTDIR)/python/lib/python$(PYTHON_VERSION); \
+-	    mkdir -p $${installdir}; \
+-	    echo "Created $${installdir}"; \
+-	    cp -f $(PYTHON_MODULE) $${installdir}/; \
+-	    pushd $${installdir} > /dev/null; \
+-	    ln -f -s $(PYTHON_MODULE) $(PYTHON_SONAME); \
+-	    popd > /dev/null; \
+-	    echo "Copied Python module to $${installdir}"; \
++	    echo "Copied libopenvdb_viewer to $(DESTDIR)/$(libdir)/"; \
+ 	fi
+ 	@#
+ 	mkdir -p $(DESTDIR)/bin
+@@ -924,7 +927,7 @@
+ 	    for f in $(UNITTEST_SRC_NAMES); \
+ 	        do $(CXX) $(CXXFLAGS) -O0 \
+ 	            -MM $$f -MT `echo $$f | sed 's%\.[^.]*%.o%'` \
+-	            -isystem $(CPPUNIT_INCL_DIR) >> $(DEPEND); \
++	            -I$(CPPUNIT_INCL_DIR) >> $(DEPEND); \
+ 	    done; \
+ 	fi
+ 

diff --git a/media-gfx/openvdb/files/openvdb-3.2.0-makefile-fixes.patch b/media-gfx/openvdb/files/openvdb-3.2.0-makefile-fixes.patch
deleted file mode 100644
index 633c380..00000000
--- a/media-gfx/openvdb/files/openvdb-3.2.0-makefile-fixes.patch
+++ /dev/null
@@ -1,93 +0,0 @@
-diff --git a/Makefile b/Makefile
-index fba4fb6..0054f64 100644
---- a/Makefile
-+++ b/Makefile
-@@ -158,6 +158,9 @@ PYCONFIG_INCL_DIR := $(PYTHON_INCL_DIR)
- # The directory containing libpython
- PYTHON_LIB_DIR := $(HFS)/python/lib
- PYTHON_LIB := -lpython$(PYTHON_VERSION)
-+# The Directory to install the python module and includes to.
-+PYTHON_INSTALL_INCL_DIR := $(DESTDIR)/python/include/python$(PYTHON_VERSION)
-+PYTHON_INSTALL_LIB_DIR := $(DESTDIR)/python/lib/python$(PYTHON_VERSION)
- # The directory containing libboost_python
- BOOST_PYTHON_LIB_DIR := /rel/depot/third_party_build/boost/rhel6-1.46.1-0/lib
- BOOST_PYTHON_LIB := -lboost_python-gcc41-mt-python26-1_46_1
-@@ -179,7 +182,6 @@ PYTHON_WRAP_ALL_GRID_TYPES := no
- # (leave blank if Doxygen is unavailable)
- DOXYGEN := doxygen
- 
--
- #
- # Ideally, users shouldn't need to change anything below this line.
- #
-@@ -717,8 +719,13 @@ $(DOC_PDF): doxygen-config $(INCLUDE_NAMES) $(SRC_NAMES) $(DOC_FILES)
- 	@echo "Generating documentation because of $(list_deps)"
- 	echo -e 'OUTPUT_DIRECTORY=./doc\nGENERATE_LATEX=YES\nGENERATE_HTML=NO' \
- 	    | cat doxygen-config - | $(DOXYGEN) - $(QUIET) \
--	    && cd ./doc/latex && make refman.pdf $(QUIET) \
-+	    && cd ./doc/latex && $(MAKE) refman.pdf $(QUIET) \
- 	    && echo 'Created doc/latex/refman.pdf'
-+	@#
-+	    mkdir -p $(DESTDIR)/share/doc/openvdb/pdf; \
-+	    echo "Created $(DESTDIR)/share/doc/openvdb/pdf"; \
-+	    cp -r -f doc/latex/refman.pdf $(DESTDIR)/share/doc/openvdb/pdf/; \
-+	    echo "Copied documentation to $(DESTDIR)/share/doc/openvdb/pdf/"; \
- 
- ifneq (,$(strip $(DOXYGEN)))
- doc: $(DOC_INDEX)
-@@ -782,6 +789,22 @@ $(PYTHON_MODULE): $(LIBOPENVDB) $(PYTHON_OBJ_NAMES)
- 	    -Wl,-rpath,$(BOOST_PYTHON_LIB_DIR) -L$(BOOST_PYTHON_LIB_DIR) $(BOOST_PYTHON_LIB) \
- 	    $(LIBOPENVDB_RPATH) -L$(CURDIR) $(LIBOPENVDB) \
- 	    $(LIBS_RPATH) $(CONCURRENT_MALLOC_LIB)
-+	@echo "Installing the python module and includes"
-+	@#
-+	    mkdir -p $(PYTHON_INSTALL_INCL_DIR); \
-+	    echo "Created $(PYTHON_INSTALL_INCL_DIR)"; \
-+	    cp -f $(PYTHON_PUBLIC_INCLUDE_NAMES) $(PYTHON_INSTALL_INCL_DIR)/; \
-+	    echo "Copied Python header files to $(PYTHON_INSTALL_INCL_DIR)"; \
-+	    mkdir -p $(PYTHON_INSTALL_LIB_DIR); \
-+	    echo "Created $(PYTHON_INSTALL_LIB_DIR)"; \
-+	    cp -f $(PYTHON_MODULE) $(PYTHON_INSTALL_LIB_DIR)/; \
-+	    pushd $(PYTHON_INSTALL_LIB_DIR) > /dev/null; \
-+	    ln -f -s $(PYTHON_MODULE) $(PYTHON_SONAME); \
-+	    popd > /dev/null; \
-+	    echo "Copied Python module to $(PYTHON_INSTALL_LIB_DIR)"
-+	@echo "Clean the Python modules so the next one will compile"
-+	@#
-+	    $(RM) $(PYTHON_OBJ_NAMES)
- 
- ifeq (yes,$(has_python))
- ifneq (,$(strip $(EPYDOC)))
-@@ -833,7 +856,7 @@ test:
- 	@echo "$@"': $$(CPPUNIT_INCL_DIR) is undefined'
- endif
- 
--install: lib python vdb_print vdb_render vdb_view doc pydoc
-+install: lib vdb_print vdb_render vdb_view doc pydoc
- 	mkdir -p $(DESTDIR)/include/openvdb
- 	@echo "Created $(DESTDIR)/include/openvdb"
- 	pushd $(DESTDIR)/include/openvdb > /dev/null; \
-@@ -870,23 +893,6 @@ install: lib python vdb_print vdb_render vdb_view doc pydoc
- 	    echo "Copied libopenvdb_viewer to $(DESTDIR)/lib/"; \
- 	fi
- 	@#
--	if [ -f $(PYTHON_MODULE) ]; \
--	then \
--	    installdir=$(DESTDIR)/python/include/python$(PYTHON_VERSION); \
--	    mkdir -p $${installdir}; \
--	    echo "Created $${installdir}"; \
--	    cp -f $(PYTHON_PUBLIC_INCLUDE_NAMES) $${installdir}/; \
--	    echo "Copied Python header files to $${installdir}"; \
--	    installdir=$(DESTDIR)/python/lib/python$(PYTHON_VERSION); \
--	    mkdir -p $${installdir}; \
--	    echo "Created $${installdir}"; \
--	    cp -f $(PYTHON_MODULE) $${installdir}/; \
--	    pushd $${installdir} > /dev/null; \
--	    ln -f -s $(PYTHON_MODULE) $(PYTHON_SONAME); \
--	    popd > /dev/null; \
--	    echo "Copied Python module to $${installdir}"; \
--	fi
--	@#
- 	mkdir -p $(DESTDIR)/bin
- 	@echo "Created $(DESTDIR)/bin/"
- 	cp -f vdb_print $(DESTDIR)/bin

diff --git a/media-gfx/openvdb/openvdb-3.2.0.ebuild b/media-gfx/openvdb/openvdb-3.2.0.ebuild
index 649cd99..ed84fad 100644
--- a/media-gfx/openvdb/openvdb-3.2.0.ebuild
+++ b/media-gfx/openvdb/openvdb-3.2.0.ebuild
@@ -2,10 +2,11 @@
 # Distributed under the terms of the GNU General Public License v2
 # $id$
 
-EAPI="6"
+EAPI=6
+
 PYTHON_COMPAT=( python{2_7,3_4,3_5} )
 
-inherit eutils python-r1
+inherit eutils python-r1 toolchain-funcs
 
 DESCRIPTION="Libs for the efficient manipulation of volumetric data"
 HOMEPAGE="http://www.openvdb.org"
@@ -43,9 +44,10 @@ DEPEND="${RDEPEND}
 
 S="${WORKDIR}/${P}/${PN}"
 
-PATCHES=( "${FILESDIR}"/${P}-numpy-fix.patch
-	  "${FILESDIR}"/${P}-makefile-fixes.patch
-	  "${FILESDIR}"/${P}-build-fixes.patch )
+PATCHES=(
+	"${FILESDIR}"/${P}-numpy-fix.patch
+	"${FILESDIR}"/${P}-fix-build-system.patch
+)
 
 python_module_compile() {
 	mypythonargs=(
@@ -54,15 +56,15 @@ python_module_compile() {
 		PYCONFIG_INCL_DIR="$(python_get_includedir)"
 		PYTHON_LIB_DIR="$(python_get_library_path)"
 		PYTHON_LIB="$(python_get_LIBS)"
-		PYTHON_INSTALL_INCL_DIR="${D}$(python_get_includedir)"
-		PYTHON_INSTALL_LIB_DIR="${D}$(python_get_sitedir)"
+		PYTHON_INSTALL_INCL_DIR="${D%/}$(python_get_includedir)"
+		PYTHON_INSTALL_LIB_DIR="${D%/}$(python_get_sitedir)"
 		NUMPY_INCL_DIR="$(python_get_sitedir)"/numpy/core/include/numpy
 		BOOST_PYTHON_LIB_DIR="${myprefixlibdir}"
 		BOOST_PYTHON_LIB=-lboost_python-${EPYTHON/python/}
 	)
 
 	einfo "Compiling module for ${EPYTHON}."
-	emake python "${myemakeargs[@]}" "${mypythonargs[@]}" EPYDOC=""
+	emake python "${myemakeargs[@]}" "${mypythonargs[@]}" EPYDOC="" CXX="$(tc-getCXX)"
 
 	# This is so the correct version of pdoc is used
 	mypyscriptdir=$(python_get_scriptdir)
@@ -84,7 +86,9 @@ src_install() {
 	# So individule targets can be called without duplication
 	# Common depends:
 	local myemakeargs=(
-		rpath=no shared=yes
+		rpath=no
+		shared=yes
+		libdir="$(get_libdir)"
 		LIBOPENVDB_RPATH=
 		DESTDIR="${D}"
 		HFS="${myprefix}"
@@ -154,5 +158,5 @@ src_install() {
 	fi
 
 	einfo "Compiling the main components."
-	emake install "${myemakeargs[@]}" "${mypythonargs[@]}"
+	emake install "${myemakeargs[@]}" "${mypythonargs[@]}" CXX="$(tc-getCXX)"
 }


             reply	other threads:[~2016-10-27 18:34 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-10-27 18:34 David Seifert [this message]
  -- strict thread matches above, loose matches on Subject: below --
2017-01-15 13:34 [gentoo-commits] repo/gentoo:master commit in: media-gfx/openvdb/, media-gfx/openvdb/files/ David Seifert
2018-08-04 18:21 Jonathan Scruggs
2018-08-14 11:39 Jonathan Scruggs
2018-08-15 20:08 Jonathan Scruggs
2018-08-28 14:43 Jonathan Scruggs
2018-08-28 14:43 Jonathan Scruggs
2020-08-24 13:42 Sam James
2020-08-24 13:42 Sam James
2020-08-24 13:42 Sam James
2020-08-24 13:42 Sam James
2021-03-14  0:25 Andreas Sturmlechner
2021-07-09  3:11 Ionen Wolkens
2021-10-31  2:19 Sam James
2021-10-31  2:19 Sam James
2021-12-02  2:34 Sam James
2021-12-10  4:31 Sam James
2022-02-15 19:51 Sam James
2022-02-18 22:23 Yixun Lan
2022-09-25  1:27 Sam James
2023-01-12 13:08 Joonas Niilola

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1477593251.f0504839a891e2a6fa94dd3da3e520c8b9dc8cb3.soap@gentoo \
    --to=soap@gentoo.org \
    --cc=gentoo-commits@lists.gentoo.org \
    --cc=gentoo-dev@lists.gentoo.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox