| Summary: | dev-python/pythonmagick-0.9.12 fails to build with boost-1.60.0 | ||
|---|---|---|---|
| Product: | Gentoo Linux | Reporter: | Helmut Jarausch <jarausch> |
| Component: | Current packages | Assignee: | Python Gentoo Team <python> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | Martin.vGagern, mmokrejs |
| Priority: | Normal | ||
| Version: | unspecified | ||
| Hardware: | All | ||
| OS: | Linux | ||
| URL: | https://github.com/ImageMagick/PythonMagick/issues/1 | ||
| Whiteboard: | |||
| Package list: | Runtime testing required: | --- | |
| Attachments: |
build log
Remove methods using void* arguments |
||
|
Description
Helmut Jarausch
2016-04-13 16:27:42 UTC
Did some trial and error. In _Image.cpp, the problematic lines apparently are the following two:
scope* Magick_Blob_scope = new scope(
class_< Magick::Blob >("Blob", init< >())
//--> .def(init< const void*, size_t >())
.def(init< const Magick::Blob& >())
.def("base64", (void (Magick::Blob::*)(const std::string) )&Magick::Blob::base64)
.def("base64", (std::string (Magick::Blob::*)() )&Magick::Blob::base64)
.def("update", &update_wrapper)
//--> .def("updateNoCopy", &Magick::Blob::updateNoCopy, Magick_Blob_updateNoCopy_overloads_2_3())
.def("length", &Magick::Blob::length)
);
But there are other problematic cases in e.g. _Image.cpp. Needs further investigation.
Created attachment 431176 [details, diff] Remove methods using void* arguments I've also suggested this approach upstream: https://github.com/ImageMagick/PythonMagick/pull/2 (In reply to Martin von Gagern from comment #2) > Created attachment 431176 [details, diff] [details, diff] > Remove methods using void* arguments > > I've also suggested this approach upstream: > https://github.com/ImageMagick/PythonMagick/pull/2 Many thanks, it works fine here, too. Thanks, the patch works for me as well. Upstream 0.9.13 includes https://github.com/ImageMagick/PythonMagick/commit/659906f137907fb2f7569eac5cfe7ed2b28cd179 which should fix this. As far as I can tell it removes some of the methods I removed, and provides wrappers for some others to use std::string instead of void* and size_t. Fixed in 0.9.14 in the tree. |