Opened 12 years ago
Closed 4 years ago
#35307 closed defect (wontfix)
opencv @2.4.2 +universal configure failure: command execution failed
Reported by: | lucasvickers@… | Owned by: | stromnov (Andrey Stromnov) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | 2.1.1 |
Keywords: | haspatch | Cc: | BasicBaer (Hendrik T. Voelker), pjue2009@…, ryandesign (Ryan Carsten Schmidt), mkae (Marko Käning) |
Port: | opencv |
Description (last modified by ryandesign (Ryan Carsten Schmidt))
Problem appears to be
-- Could NOT find PythonLibs: Found unsuitable version "2.7.2", but required is at least "2.7.3"
- I have confirmed that Python 2.7.3 is installed multiple times.
- I force uninstalled python, confirmed file '/opt/local/lib/libpython2.7.dylib' was non existant, then installed python 2.7.3 +universal
- I also tried allowing opencv port to control the installation of python 2.7.3.
In every instance opencv port claims the python libs are 2.7.2. I believe this is the point of failure.
Attachments (2)
Change History (19)
Changed 12 years ago by lucasvickers@…
comment:2 Changed 12 years ago by ryandesign (Ryan Carsten Schmidt)
Cc: | lucasvickers@… removed |
---|---|
Description: | modified (diff) |
Keywords: | opencv python removed |
comment:3 Changed 12 years ago by ryandesign (Ryan Carsten Schmidt)
I can confirm that opencv +universal does not complete the configure phase on my Lion machine either. However I don't think it has anything to do with python; those python failure messages also appear when you try to install opencv without the universal variant, but that still succeeds.
comment:4 Changed 12 years ago by lucasvickers@…
You are correct I was able to compile without the +universal
Does main.log give you any further details?
thanks
comment:6 Changed 12 years ago by lucasvickers@…
Any rough eta on how long this may take or general turn around time on tickets?
Trying to decide how much effort I should put into compiling elsewhere.
Thanks
comment:8 follow-up: 10 Changed 12 years ago by BasicBaer (Hendrik T. Voelker)
It seems that the problem are not the Python libs. The message is just misleading, as the later following CMake code will pick up the correct library.
The problem is that +universal will cause the removal of one include file from the main CMakeLists.txt file which contains a macro that is required later on. While the compile will fail when the include file cmake/OpenCVPCHSupport.cmake is used as is, as it enables PCH, the simple solution is to add a line containing
SET(PCHSupport_FOUND FALSE)
directly after the include. I have attached the new diff file that replaces the old one and does as described above.
Changed 12 years ago by BasicBaer (Hendrik T. Voelker)
Attachment: | patch-pch-CMakeLists.txt.diff added |
---|
updates patch file
comment:9 Changed 12 years ago by jmroot (Joshua Root)
Keywords: | haspatch added |
---|
comment:10 follow-up: 11 Changed 12 years ago by ryandesign (Ryan Carsten Schmidt)
Cc: | ryandesign@… added |
---|
Replying to basicbaer@…:
the simple solution is to add a line containing
SET(PCHSupport_FOUND FALSE)
directly after the include
I don't really understand the problem or the proposed solution so I'm a little uncomfortable committing it. I'd feel better if you could show me a statement from the developers of OpenCV stating that this is the correct solution. Or point me to where they've already committed this fix to their repository. If they haven't done that yet, then could you please first work with the developers of OpenCV to get this or some other solution accepted and committed to their repository? Then we can apply the same fix in MacPorts, until they release a new version of OpenCV.
comment:11 follow-up: 12 Changed 12 years ago by BasicBaer (Hendrik T. Voelker)
Replying to ryandesign@…:
I don't really understand the problem or the proposed solution so I'm a little uncomfortable committing it.
I don't understand either why the Universal variant cannot be build with PCH support.
I'd feel better if you could show me a statement from the developers of OpenCV
My patch has nothing to do with the OpenCV developers. I was just replacing the patch file ports/graphics/opencv/files/patch-pch-CMakeLists.txt.diff that is already part of the port. So I fixed the port, not OpenCV.
If they haven't done that yet, then could you please first work with the developers of OpenCV
Honestly I don't feel that as my responsibility as I am not the maintainer of this port.
For me the fix works and I can live with a few port files maintained by myself locally ...
comment:12 Changed 12 years ago by ryandesign (Ryan Carsten Schmidt)
Replying to basicbaer@…:
If they haven't done that yet, then could you please first work with the developers of OpenCV
Honestly I don't feel that as my responsibility as I am not the maintainer of this port.
Well clearly nobody has felt it is their responsibility, since nobody has agreed to be this port's maintainer. :)
comment:13 Changed 12 years ago by jmroot (Joshua Root)
Owner: | changed from macports-tickets@… to stromnov@… |
---|
comment:14 Changed 8 years ago by mkae (Marko Käning)
Cc: | mkae added |
---|
Shouldn't this be closed as wontfix as opencv has advanced in the meantime big time?
comment:15 Changed 4 years ago by mascguy (Christopher Nielsen)
Given that we don't support opencv 2.x, can this be closed?
comment:16 Changed 4 years ago by mascguy (Christopher Nielsen)
This ticket is no longer relevant, as we don't have an OpenCV 2 port. Can someone close?
comment:17 Changed 4 years ago by mf2k (Frank Schima)
Resolution: | → wontfix |
---|---|
Status: | new → closed |
output from opencv build