openMVG + VS2013 build error

Created on 14 Nov 2016 · 14Comments · Source: openMVG/openMVG

Hi,
thanks very much for providing the excellent SfM tools. Recently, I have pull the latest source code and build it under vs2013 on windows 7 x64 platform. However, the project named "openMVG_system" could not pass built process and some errors have been reported as following. After search these errors, some conflicts maybe existed between C99 and C++11 (http://stackoverflow.com/questions/25561677/intmax-max-was-not-declared-in-this-scope).

Source

json87

Most helpful comment

I've had the same issue building on VS2013, so I made VS2013-specific minimal intrusion changes. openMVG builds in its entirety, including unit tests, and unit tests run without errors.

As it is our need, we will be maintaining VS2013 compatibility the foreseeable future on our branch at:
https://github.com/modriplanetdoo/openMVG/tree/vs2013

TheDoubleB on 6 Feb 2017

👍2

All 14 comments

Use Visual studio 2015 instead.

whuaegeanse on 14 Nov 2016

👍1

Please use VisualStudio as advised in BUILD file
I think you could manage to build for VS2013 but you will need to make some minor code change.

pmoulon on 15 Nov 2016

@json87 Any feedback?

pmoulon on 16 Nov 2016

yes, I am testing the solution under vs2013. I would give my feedback later today. @pmoulon

json87 on 17 Nov 2016

good news! I have successfully built openMVG with VS 2013 under windows 7 platform. The errors listed above could be dealt by adding one macro definition in the CMakeFileList.txt in the src folder. I am a freshman for openMVG. then further tests have not been conducted. @pmoulon

IF (WIN32)
ADD_DEFINITIONS(-DNOMINMAX)
ADD_DEFINITIONS(-DVC_EXTRALEAN)
IF (MSVC)
ADD_DEFINITIONS(/bigobj)
_ADD_DEFINITIONS(-D__STDC_LIMIT_MACROS)_
ENDIF (MSVC)
ENDIF (WIN32)

json87 on 17 Nov 2016

You must try to build everything (including enable the unit test and samples in the cmakelist)
and then test the run-test target to ensure that OpenMVG will work as it must for your compiler and machine.

pmoulon on 18 Nov 2016

yes, openMVG core libraries, samples and main apps have been compiled successfuly!

[email protected]

From: Pierre Moulon
Date: 2016-11-18 09:35
To: openMVG/openMVG
CC: json87; Mention
Subject: Re: [openMVG/openMVG] openMVG + VS2013 build error (#676)
You must try to build everything (including enable the unit test and samples in the cmakelist)
and then test the run-test target to ensure that OpenMVG will work as it must for your compiler and machine.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

json87 on 18 Nov 2016

Did you checked that unit test was running fine?

pmoulon on 23 Nov 2016

Hello all, I am trying to do the same thing in vs2013, and get the error message:

CMake Error at cameras/CMakeLists.txt:2 (UNIT_TEST):
Unknown CMake command "UNIT_TEST".

unfortunately, i got the same message with 2015. how can I fix it?

my platform is windows 10 x64

robin32205 on 23 Jan 2017

@robin32205 Using VS2015 is advised in order to have a full C++11 feature support. https://github.com/openMVG/openMVG/blob/master/BUILD#L23

@robin32205 Can you show the full cmake log?
Continuous integration on VS2015 was ok for v1.1 as you see here https://ci.appveyor.com/project/pmoulon/openmvg/build/1.0.302

pmoulon on 23 Jan 2017

Hi, please check the attachment for info, thank you!

寄件者: Pierre Moulon notifications@github.com
寄件日期: 2017年1月23日上午 11:43
收件者: openMVG/openMVG
副本: robin32205; Mention
主旨: Re: [openMVG/openMVG] openMVG + VS2013 build error (#676)

@robin32205https://github.com/robin32205 Using VS2015 is advised in order to have a full C++11 feature support. https://github.com/openMVG/openMVG/blob/master/BUILD#L23

@robin32205https://github.com/robin32205 Can you show the full cmake log?
Continuous integration on VS2015 was ok for v1.1 as you see here https://ci.appveyor.com/project/pmoulon/openmvg/build/1.0.302

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/openMVG/openMVG/issues/676#issuecomment-274393242, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AWIS6pawZqZXREIdlSRQDO_DT_waQonmks5rVCHJgaJpZM4KxAW8.

robin32205 on 23 Jan 2017

Please use the github online interface for dropping attachment.

pmoulon on 23 Jan 2017

I've had the same issue building on VS2013, so I made VS2013-specific minimal intrusion changes. openMVG builds in its entirety, including unit tests, and unit tests run without errors.

As it is our need, we will be maintaining VS2013 compatibility the foreseeable future on our branch at:
https://github.com/modriplanetdoo/openMVG/tree/vs2013

TheDoubleB on 6 Feb 2017

👍2

I'm closing the issue since various solution have been provided: