Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[OpenCV2] add library for projects that require the C API #3360

Closed
wants to merge 5 commits into from

Conversation

cenit
Copy link
Contributor

@cenit cenit commented Apr 27, 2018

as per title, I propose the inclusion of the older OpenCV 2, guarded with an error like MySQL/MariaDB, since it is necessary to build projects that still use the C API (abandoned/broken in OpenCV3). For example, it is necessary to build YOLO on Windows ;)

@cenit cenit changed the title [OpenCV2] add library for older C projects [OpenCV2] add library for projects that require the C API Apr 28, 2018
@cenit
Copy link
Contributor Author

cenit commented Apr 28, 2018

I did another commit, to fix building on macOS (I hope it fixes also Ubuntu, still not tested). Since vcpkg is now cross-platform, I hope it is better now 🤓

@cenit
Copy link
Contributor Author

cenit commented Apr 28, 2018

I tested building also on Ubuntu and it worked. Added a small notice (comment) at the beginning of the port file to explain pre-requisites.

@cenit
Copy link
Contributor Author

cenit commented May 3, 2018

As a side note, I tried building also on x86-windows but I got a failure when linking with OpenGL32.lib, with the linker complaining that it couldn't solve _imp_wglGetProcAddress symbol...

@cenit
Copy link
Contributor Author

cenit commented Nov 24, 2018

I think that this port is in a good shape to be at least tried by someone, if interested @ras0219-msft @UnaNancyOwen @patrikhuber @jasjuang

@patrikhuber
Copy link
Contributor

@cenit Thanks, it might become useful some day (as you mentioned... YOLO & stuff...), but for now I long moved to OpenCV3 (and probably 4 soon) for my own projects :-)

@jasjuang
Copy link
Contributor

@cenit Thanks but I have move forwarded to OpenCV3 a few years ago for my projects so I won't be able to test it.

@cenit cenit mentioned this pull request Feb 27, 2019
@cenit
Copy link
Contributor Author

cenit commented Apr 9, 2019

closed because there's not enough interest and it would be difficult to keep tested (vcpkg discourages not using the latest version by design)

@cenit cenit closed this Apr 9, 2019
@cenit cenit deleted the dev/cenit/opencv2 branch May 20, 2019 07:37
@cenit cenit restored the dev/cenit/opencv2 branch August 23, 2019 06:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants