Build errors

Question

Build errors

michaelitvin opened this issue 7 years ago · comments

I'm trying to build inpaint in Xcode 8 with C++11 and OpenCV 2.3.1.
Added the files from /src to the project and compiled. Getting many errors that seem related to OpenCV compatibility:

Michael Litvin commented 7 years ago

Thanks

Michael Litvin commented 7 years ago

Thanks

Christoph Heindl · Answer 1 · Thu Feb 09 2017 17:32:55 GMT+0800 (China Standard Time)

You are using quite an outdated OpenCV. Any chance you could try to use a newer version? Michael Litvin <notifications@github.com> schrieb am Mi., 8. Feb. 2017 um 17:46 Uhr:

…

I'm trying to build inpaint in Xcode 8 with C++11 and OpenCV 2.3.1. Added the files from /src to the project and compiled. Getting many errors that seem related to OpenCV compatibility: [image: image] <https://cloud.githubusercontent.com/assets/5813969/22746879/ffcf987a-ee2d-11e6-97f3-8e06d9374d04.png> — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#7>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AACWpSUXN4EJxiUwmguAFsiDvhIowvD1ks5rafF7gaJpZM4L7ELV> .

Michael Litvin · Answer 2 · Thu Feb 09 2017 17:34:32 GMT+0800 (China Standard Time)

Which versions should be supported?

Christoph Heindl · Answer 3 · Thu Feb 09 2017 17:56:52 GMT+0800 (China Standard Time)

2.4, 3.1 Michael Litvin <notifications@github.com> schrieb am Do., 9. Feb. 2017 um 10:34 Uhr:

…

What versions should be supported? — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#7 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AACWpYM8CVKvePeo_tV0ObpEm_AwTHDtks5rat2pgaJpZM4L7ELV> .

Christoph Heindl · Answer 4 · Fri Feb 10 2017 02:05:24 GMT+0800 (China Standard Time)

Works as expected? Michael Litvin <notifications@github.com> schrieb am Do., 9. Feb. 2017 um 18:47 Uhr:

…

Thanks — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#7 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AACWpVzTknH73buaNPj_fMqIkdXcYhgwks5ra1ETgaJpZM4L7ELV> .

Christoph Heindl · Answer 5 · Sun Feb 12 2017 14:24:51 GMT+0800 (China Standard Time)

Assuming to be fixed.

Michael Litvin · Answer 6 · Sun Feb 12 2017 22:22:04 GMT+0800 (China Standard Time)

Updated to 3.2.
Most errors are fixed, except this error in meanshift.cpp:

I believe that happens because seeds.row(count) is a temporary and therefore should be const. If I add const on line 34 it compiles fine:

Christoph Heindl · Answer 7 · Sun Feb 12 2017 22:58:35 GMT+0800 (China Standard Time)

ok, thanks. I've changed that in master. Note, meanshift.cpp is currently not used by CMakeLists.txt which is why the error hasn't popped up earlier. How did you generate your solution file for build? Best, Christoph Michael Litvin <notifications@github.com> schrieb am So., 12. Feb. 2017 um 15:22 Uhr:

…

Updated to 3.2. Most errors are fixed, except this error in meanshift.cpp: [image: image] <https://cloud.githubusercontent.com/assets/5813969/22862776/d0026556-f13d-11e6-80b4-e707ccd76076.png> I believe that happens because seeds.row(count) is a temporary and therefore should be const. If I add const on line 34 it compiles fine: [image: image] <https://cloud.githubusercontent.com/assets/5813969/22862848/52e2163c-f13f-11e6-8349-d7a1177c08a5.png> — You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub <#7 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AACWpc1RaU5dT9iEaMqwfvL39G_2y34Wks5rbxWMgaJpZM4L7ELV> .

Michael Litvin · Answer 8 · Sun Feb 12 2017 23:00:32 GMT+0800 (China Standard Time)

I haven't used CMake, just added all the cpp files to a project. Should I remove meanshift and use only the methods that are in CMakeLists.txt?

Christoph Heindl · Answer 9 · Sun Feb 12 2017 23:01:30 GMT+0800 (China Standard Time)

No its fine as long as it compiles. Best, Christoph Michael Litvin <notifications@github.com> schrieb am So., 12. Feb. 2017 um 16:00 Uhr:

…

I haven't used CMake, just added all the cpp files to a project. Should I remove meanshift? — You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub <#7 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AACWpV6DJPGokNtTc2galujhOAlzeseIks5rbx6QgaJpZM4L7ELV> .