Skip to content

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

Closed
cenit wants to merge 5 commits intomicrosoft:masterfrom
cenit:dev/cenit/opencv2
Closed

[OpenCV2] add library for projects that require the C API#3360
cenit wants to merge 5 commits intomicrosoft:masterfrom
cenit:dev/cenit/opencv2

Conversation

@cenit
Copy link
Copy Markdown
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
Copy Markdown
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 cenit force-pushed the dev/cenit/opencv2 branch from 14b5a89 to ef850ed Compare April 28, 2018 17:14
@cenit
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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