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)

main.log (94.8 KB) - added by lucasvickers@… 12 years ago.
output from opencv build
patch-pch-CMakeLists.txt.diff (349 bytes) - added by BasicBaer (Hendrik T. Voelker) 12 years ago.
updates patch file

Download all attachments as: .zip

Change History (19)

Changed 12 years ago by lucasvickers@…

Attachment: main.log added

output from opencv build

comment:1 Changed 12 years ago by lucasvickers@…

Cc: lucasvickers@… added

Cc Me!

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:5 Changed 12 years ago by BasicBaer (Hendrik T. Voelker)

Cc: basicbaer@… added

Cc Me!

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:7 Changed 12 years ago by pjue2009@…

Cc: pjue2009@… added

Cc Me!

comment:8 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)

updates patch file

comment:9 Changed 12 years ago by jmroot (Joshua Root)

Keywords: haspatch added

comment:10 in reply to:  8 ; 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 in reply to:  10 ; 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 in reply to:  11 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: newclosed
Note: See TracTickets for help on using tickets.