Re: [osg-users] Preparing to make 3.5.3 dev release, please test
Hi, Tried compiling my own project with the latest master got some more warnings coming from included header files in OSG: Code: OpenSceneGraph\include\osgShadow/ViewDependentShadowTechnique(209): warning C4100: 'maxSize': unreferenced formal parameter OpenSceneGraph\include\osg/GLObjects(64): warning C4100: 'fs': unreferenced formal parameter OpenSceneGraph\include\osg/GLObjects(67): warning C4100: 'out': unreferenced formal parameter OpenSceneGraph\include\osg/GLObjects(68): warning C4100: 'out': unreferenced formal parameter Regards Björn[/code] -- Read this topic online here: http://forum.openscenegraph.org/viewtopic.php?p=67466#67466 ___ osg-users mailing list osg-users@lists.openscenegraph.org http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org
Re: [osg-users] Preparing to make 3.5.3 dev release, please test
Hi Laurens, Do you have a githib account that you'd like me to credit with this submissions? Where possible I've been utilizing the git commit --author feature where I have the email address used on github. Robert. On 6 June 2016 at 15:21, Voerman, L. wrote: > Hi Robbert, Björn, > as I am updating to visual studio 2015 too, I added the new FBX sdk for > 2015, and get a single warning: > > 54>OpenSceneGraph\src\osgPlugins\fbx\WriterCompareTriangle.cpp(23): warning > C4456: declaration of 'g' hides previous local declaration > 54> OpenSceneGraph\src\osgPlugins\fbx\WriterCompareTriangle.cpp(14): note: > see declaration of 'g' > > attached are a fix for the warning and an update for the FindFBX.cmake to > find the new SDK for windows. > Regards, Laurens. > > On Mon, Jun 6, 2016 at 3:41 PM, Robert Osfield > wrote: >> >> Hi Björn, >> >> On 6 June 2016 at 13:55, Björn Blissing wrote: >> > In the LUA-plugin: >> > >> > >> > Code: >> > >> > >> > d:\code\github\openscenegraph\src\osgplugins\lua\lua-5.2.3\src\lapi.c(1110): >> > warning C4702: unreachable code >> >> Removing the return statement that the warning is complaining about >> would force a in the break method API. Could be a slippery slope of >> changes required so I've just disabled the warning, at least tried to >> using simple CMake entry to what you added to the 3ds plugin. >> >> > In the OSC-plugin: >> > >> > >> > Code: >> > openscenegraph\src\osgPlugins\osc\ip\win32\NetworkingUtils.cpp(80): >> > warning C4996: 'gethostbyname': Use getaddrinfo() or GetAddrInfoW() instead >> > or define _WINSOCK_DEPRECATED_NO_WARNINGS to disable deprecated API >> > warnings >> > C:\Program Files (x86)\Windows Kits\8.1\Include\um\winsock2.h(2238): >> > note: see declaration of 'gethostbyname' >> > openscenegraph\src\osgPlugins\osc\ip\win32\UdpSocket.cpp(401): warning >> > C4456: declaration of 'currentTimeMs' hides previous local declaration >> > openscenegraph\src\osgPlugins\osc\ip\win32\UdpSocket.cpp(386): note: >> > see declaration of 'currentTimeMs' >> > >> > The first one is self explainatory. The second one is a simple variable >> > shadowing. Which I do not think have any unintended consequences. >> >> This is 3rd party code that's integrated, we could fix or just disable >> the warnings. If we modify the code we'd need to test it to make >> sure. I'm not the original author of the code or a Windows hack so >> I'll not personally dive in and start hacking with the code. If no >> one else feels the urge then I'd suggest we just disable the warning. >> >> The shadowing warning I've simply removed the second double as it's not >> needed. >> >> >> > In the OSG-plugin: >> > >> > >> > Code: >> > openscenegraph\src\osgPlugins\osg\ReaderWriterOSG.cpp(257): warning >> > C4459: declaration of 'NodeList' hides global declaration >> > openscenegraph\include\osg/Group(22): note: see declaration of >> > 'osg::NodeList' >> >> I've changed the code to simply use the osg::NodeList variant. >> >> >> > And finally in the TXP plugin: >> > >> > Code: >> > openscenegraph\src\osgPlugins\txp\trpage_pparse.cpp(241): warning C4458: >> > declaration of 'imageHelp' hides class member >> > openscenegraph\src\osgPlugins\txp\trpage_print.h(136): note: see >> > declaration of 'trpgPrintGraphParser::imageHelp' >> > >> > This last one may actually have consequences which the author did not >> > intend. But I am entirely not sure. The author have a protected member >> > variable named imageHelp, which is set in the constructor. It also have an >> > access method. But the variable is not used anywhere inside of the class, >> > instead the scoped variable with the same name is used. >> >> I've changed the local name to parse_imageHelp to make sure it's using >> the local one. >> >> With these fixes now checked into master it should just be the OSC warning >> left. >> >> Robert. >> ___ >> osg-users mailing list >> osg-users@lists.openscenegraph.org >> http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org > > > > ___ > osg-users mailing list > osg-users@lists.openscenegraph.org > http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org > ___ osg-users mailing list osg-users@lists.openscenegraph.org http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org
Re: [osg-users] Moved to BUILD FORUM - Best Pipeline For Integrating Blender Models?
Hi, Christian, that is useful. Ty sir. I'll try osgconv in a similar workflow. Thank you! Cheers, Dave -- Read this topic online here: http://forum.openscenegraph.org/viewtopic.php?p=67464#67464 ___ osg-users mailing list osg-users@lists.openscenegraph.org http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org
Re: [osg-users] Preparing to make 3.5.3 dev release, please test
Hi Björn, On 6 June 2016 at 13:55, Björn Blissing wrote: > In the LUA-plugin: > > > Code: > > d:\code\github\openscenegraph\src\osgplugins\lua\lua-5.2.3\src\lapi.c(1110): > warning C4702: unreachable code Removing the return statement that the warning is complaining about would force a in the break method API. Could be a slippery slope of changes required so I've just disabled the warning, at least tried to using simple CMake entry to what you added to the 3ds plugin. > In the OSC-plugin: > > > Code: > openscenegraph\src\osgPlugins\osc\ip\win32\NetworkingUtils.cpp(80): warning > C4996: 'gethostbyname': Use getaddrinfo() or GetAddrInfoW() instead or define > _WINSOCK_DEPRECATED_NO_WARNINGS to disable deprecated API warnings > C:\Program Files (x86)\Windows Kits\8.1\Include\um\winsock2.h(2238): note: > see declaration of 'gethostbyname' > openscenegraph\src\osgPlugins\osc\ip\win32\UdpSocket.cpp(401): warning C4456: > declaration of 'currentTimeMs' hides previous local declaration > openscenegraph\src\osgPlugins\osc\ip\win32\UdpSocket.cpp(386): note: see > declaration of 'currentTimeMs' > > The first one is self explainatory. The second one is a simple variable > shadowing. Which I do not think have any unintended consequences. This is 3rd party code that's integrated, we could fix or just disable the warnings. If we modify the code we'd need to test it to make sure. I'm not the original author of the code or a Windows hack so I'll not personally dive in and start hacking with the code. If no one else feels the urge then I'd suggest we just disable the warning. The shadowing warning I've simply removed the second double as it's not needed. > In the OSG-plugin: > > > Code: > openscenegraph\src\osgPlugins\osg\ReaderWriterOSG.cpp(257): warning C4459: > declaration of 'NodeList' hides global declaration > openscenegraph\include\osg/Group(22): note: see declaration of > 'osg::NodeList' I've changed the code to simply use the osg::NodeList variant. > And finally in the TXP plugin: > > Code: > openscenegraph\src\osgPlugins\txp\trpage_pparse.cpp(241): warning C4458: > declaration of 'imageHelp' hides class member > openscenegraph\src\osgPlugins\txp\trpage_print.h(136): note: see > declaration of 'trpgPrintGraphParser::imageHelp' > > This last one may actually have consequences which the author did not intend. > But I am entirely not sure. The author have a protected member variable named > imageHelp, which is set in the constructor. It also have an access method. > But the variable is not used anywhere inside of the class, instead the scoped > variable with the same name is used. I've changed the local name to parse_imageHelp to make sure it's using the local one. With these fixes now checked into master it should just be the OSC warning left. Robert. ___ osg-users mailing list osg-users@lists.openscenegraph.org http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org
Re: [osg-users] [build] DAE build failure.. dae.h not found
rmilh wrote: > > > > Yes. In FindCOLLADA.cmake : > > IF (COLLADA_INCLUDE_DOMANY_DIR) > SET(COLLADA_DOM_2_4_OR_LATER TRUE) > ELSEIF() > SET(COLLADA_DOM_2_4_OR_LATER FALSE) > ENDIF() > > You will have to unset/remove entry COLLADA_INCLUDE_DOMANY_DIR . > > Cheers, > Robert Milharcic > You are my saviour! Thank you my good man. I now can compile the dae plugin. -- Read this topic online here: http://forum.openscenegraph.org/viewtopic.php?p=67459#67459 ___ osg-users mailing list osg-users@lists.openscenegraph.org http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org
Re: [osg-users] Preparing to make 3.5.3 dev release, please test
robertosfield wrote: > The HORROR!!! :-) What? Don't you approve of using the variable name 'c' throughout the entire codebase and in pretty much every scope? :-) With these warnings out of the way I only get these warnings: In the LUA-plugin: Code: d:\code\github\openscenegraph\src\osgplugins\lua\lua-5.2.3\src\lapi.c(1110): warning C4702: unreachable code Which is also denoted by a corresponding comment in the code: "/* code unreachable; will unlock when control actually leaves the kernel */" So this warning should probably be disabled. In the OSC-plugin: Code: openscenegraph\src\osgPlugins\osc\ip\win32\NetworkingUtils.cpp(80): warning C4996: 'gethostbyname': Use getaddrinfo() or GetAddrInfoW() instead or define _WINSOCK_DEPRECATED_NO_WARNINGS to disable deprecated API warnings C:\Program Files (x86)\Windows Kits\8.1\Include\um\winsock2.h(2238): note: see declaration of 'gethostbyname' openscenegraph\src\osgPlugins\osc\ip\win32\UdpSocket.cpp(401): warning C4456: declaration of 'currentTimeMs' hides previous local declaration openscenegraph\src\osgPlugins\osc\ip\win32\UdpSocket.cpp(386): note: see declaration of 'currentTimeMs' The first one is self explainatory. The second one is a simple variable shadowing. Which I do not think have any unintended consequences. In the OSG-plugin: Code: openscenegraph\src\osgPlugins\osg\ReaderWriterOSG.cpp(257): warning C4459: declaration of 'NodeList' hides global declaration openscenegraph\include\osg/Group(22): note: see declaration of 'osg::NodeList' This class defines the following: typedef std::vector NodeList; Which is different compared to the definintion in osg/Group, which uses a ref_ptr instead. And finally in the TXP plugin: Code: openscenegraph\src\osgPlugins\txp\trpage_pparse.cpp(241): warning C4458: declaration of 'imageHelp' hides class member openscenegraph\src\osgPlugins\txp\trpage_print.h(136): note: see declaration of 'trpgPrintGraphParser::imageHelp' This last one may actually have consequences which the author did not intend. But I am entirely not sure. The author have a protected member variable named imageHelp, which is set in the constructor. It also have an access method. But the variable is not used anywhere inside of the class, instead the scoped variable with the same name is used. Regards Björn -- Read this topic online here: http://forum.openscenegraph.org/viewtopic.php?p=67457#67457 ___ osg-users mailing list osg-users@lists.openscenegraph.org http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org
Re: [osg-users] [build] DAE build failure.. dae.h not found
Hi Dave Hi, This is my current latest COLLADA configuration. The reason that the COLLADA_DOM_ROOT was wrong was because I had an improper COLLADA_DIR environment variable. https://drive.google.com/open?id=0BzUf-8Ad-iIkOWhuYUhWSTZEWDQ Can you see something in this configuration that would cause COLLADA_DOM_2_4_OR_LATER to be set? Yes. In FindCOLLADA.cmake : IF (COLLADA_INCLUDE_DOMANY_DIR) SET(COLLADA_DOM_2_4_OR_LATER TRUE) ELSEIF() SET(COLLADA_DOM_2_4_OR_LATER FALSE) ENDIF() You will have to unset/remove entry COLLADA_INCLUDE_DOMANY_DIR . Cheers, Robert Milharcic ___ osg-users mailing list osg-users@lists.openscenegraph.org http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org
Re: [osg-users] Preparing to make 3.5.3 dev release, please test
The HORROR!!! :-) OK, merged the disabling of the warning for the 3ds plugin. Thanks. On 6 June 2016 at 13:02, Björn Blissing wrote: > > robertosfield wrote: >> Hi Bjorn, >> >> Could you post a log of the warnings that this CMakeLists.txt works >> around? If there aren't too many we could just fix them. >> >> Robert. > > > There are quite many, so IMHO we should just disable this warning for this > subproject. But I is your call. > > Here are the build log: > > > Code: > openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_atmosphere.c(178): warning > C4456: declaration of 'c' hides previous local declaration > openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_atmosphere.c(169): note: > see declaration of 'c' > openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_atmosphere.c(185): warning > C4456: declaration of 'c' hides previous local declaration > openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_atmosphere.c(169): note: > see declaration of 'c' > openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_atmosphere.c(204): warning > C4456: declaration of 'c' hides previous local declaration > openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_atmosphere.c(195): note: > see declaration of 'c' > openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_atmosphere.c(222): warning > C4456: declaration of 'c' hides previous local declaration > openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_atmosphere.c(213): note: > see declaration of 'c' > openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_camera.c(149): warning C4456: > declaration of 'c' hides previous local declaration > openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_camera.c(134): note: see > declaration of 'c' > openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_camera.c(155): warning C4456: > declaration of 'c' hides previous local declaration > openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_camera.c(134): note: see > declaration of 'c' > openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(721): warning C4456: > declaration of 'c' hides previous local declaration > openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(715): note: see > declaration of 'c' > openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(728): warning C4456: > declaration of 'c' hides previous local declaration > openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(715): note: see > declaration of 'c' > openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(742): warning C4456: > declaration of 'c' hides previous local declaration > openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(715): note: see > declaration of 'c' > openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(758): warning C4456: > declaration of 'c' hides previous local declaration > openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(715): note: see > declaration of 'c' > openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(776): warning C4456: > declaration of 'c' hides previous local declaration > openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(715): note: see > declaration of 'c' > openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(789): warning C4456: > declaration of 'c' hides previous local declaration > openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(715): note: see > declaration of 'c' > openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(802): warning C4456: > declaration of 'c' hides previous local declaration > openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(715): note: see > declaration of 'c' > openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(850): warning C4456: > declaration of 'c' hides previous local declaration > openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(840): note: see > declaration of 'c' > openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(859): warning C4456: > declaration of 'c' hides previous local declaration > openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(840): note: see > declaration of 'c' > openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(867): warning C4456: > declaration of 'c' hides previous local declaration > openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(840): note: see > declaration of 'c' > openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(909): warning C4456: > declaration of 'c' hides previous local declaration > openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(894): note: see > declaration of 'c' > openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_light.c(198): warning C4456: > declaration of 'c' hides previous local declaration > openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_light.c(191): note: see > declaration of 'c' > openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_light.c(205): warning C4456: > declaration of 'c' hides previous local declaration > openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_light.c(191): note: see > declaration of 'c' > openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_light.c(211): warning C4456: > declarat
Re: [osg-users] Preparing to make 3.5.3 dev release, please test
robertosfield wrote: > Hi Bjorn, > > Could you post a log of the warnings that this CMakeLists.txt works > around? If there aren't too many we could just fix them. > > Robert. There are quite many, so IMHO we should just disable this warning for this subproject. But I is your call. Here are the build log: Code: openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_atmosphere.c(178): warning C4456: declaration of 'c' hides previous local declaration openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_atmosphere.c(169): note: see declaration of 'c' openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_atmosphere.c(185): warning C4456: declaration of 'c' hides previous local declaration openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_atmosphere.c(169): note: see declaration of 'c' openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_atmosphere.c(204): warning C4456: declaration of 'c' hides previous local declaration openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_atmosphere.c(195): note: see declaration of 'c' openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_atmosphere.c(222): warning C4456: declaration of 'c' hides previous local declaration openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_atmosphere.c(213): note: see declaration of 'c' openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_camera.c(149): warning C4456: declaration of 'c' hides previous local declaration openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_camera.c(134): note: see declaration of 'c' openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_camera.c(155): warning C4456: declaration of 'c' hides previous local declaration openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_camera.c(134): note: see declaration of 'c' openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(721): warning C4456: declaration of 'c' hides previous local declaration openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(715): note: see declaration of 'c' openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(728): warning C4456: declaration of 'c' hides previous local declaration openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(715): note: see declaration of 'c' openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(742): warning C4456: declaration of 'c' hides previous local declaration openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(715): note: see declaration of 'c' openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(758): warning C4456: declaration of 'c' hides previous local declaration openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(715): note: see declaration of 'c' openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(776): warning C4456: declaration of 'c' hides previous local declaration openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(715): note: see declaration of 'c' openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(789): warning C4456: declaration of 'c' hides previous local declaration openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(715): note: see declaration of 'c' openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(802): warning C4456: declaration of 'c' hides previous local declaration openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(715): note: see declaration of 'c' openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(850): warning C4456: declaration of 'c' hides previous local declaration openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(840): note: see declaration of 'c' openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(859): warning C4456: declaration of 'c' hides previous local declaration openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(840): note: see declaration of 'c' openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(867): warning C4456: declaration of 'c' hides previous local declaration openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(840): note: see declaration of 'c' openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(909): warning C4456: declaration of 'c' hides previous local declaration openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_file.c(894): note: see declaration of 'c' openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_light.c(198): warning C4456: declaration of 'c' hides previous local declaration openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_light.c(191): note: see declaration of 'c' openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_light.c(205): warning C4456: declaration of 'c' hides previous local declaration openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_light.c(191): note: see declaration of 'c' openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_light.c(211): warning C4456: declaration of 'c' hides previous local declaration openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_light.c(191): note: see declaration of 'c' openscenegraph\src\osgPlugins\3ds\lib3ds\lib3ds_light.c(218): warning C4456: declaration of 'c' hides previous local declaration openscenegraph\src\osgPlugins\3ds\lib3ds\
Re: [osg-users] [build] DAE build failure.. dae.h not found
Hi, This is my current latest COLLADA configuration. The reason that the COLLADA_DOM_ROOT was wrong was because I had an improper COLLADA_DIR environment variable. https://drive.google.com/open?id=0BzUf-8Ad-iIkOWhuYUhWSTZEWDQ Can you see something in this configuration that would cause COLLADA_DOM_2_4_OR_LATER to be set? Sorry for the big red caps , I was happy to get to this level of understanding [Idea] , so thought I'd emphasize my current latest quandary [Exclamation] Thank you for what I hope to be the final bit of guidance! Cheers, Dave -- Read this topic online here: http://forum.openscenegraph.org/viewtopic.php?p=67450#67450 ___ osg-users mailing list osg-users@lists.openscenegraph.org http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org
Re: [osg-users] [build] DAE build failure.. dae.h not found
rmilh wrote: > > > I believe COLLADA_INCLUDE_DIR shouldn't point to DOM headers. You should > probably set it to > c:/osg/OpenSceneGraph-3.4.0/OpenSceneGraph-3.4.0/3rdParty/x64/include . You > should also set DEBUG variants of the boost libraries to point to > libboost_*mt-gd*.lib (mt = multi threaded, gd = debug) and > COLLADA_PCRECPP_LIBRARY_DEBUG to pcrecppd.lib (pcrecpp(with d suffix).lib) > and also COLLADA_PCRE_LIBRARY_DEBUG to pcred.lib. > > > > Cheers, > Robert M. > > Thank you Robert for confirming this. I finally realized this when I looked at the cmakelists. -- Read this topic online here: http://forum.openscenegraph.org/viewtopic.php?p=67447#67447 ___ osg-users mailing list osg-users@lists.openscenegraph.org http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org
Re: [osg-users] [build] DAE build failure.. dae.h not found
Hi, ... The process of documenting the details, led to a tiny bit of progress (https://drive.google.com/open?id=0BzUf-8Ad-iIkY1VnQWtKNnpKNWs). Oddly, daeRMaterials.cpp references domAny.h in dom, yet as mentioned in my detailed analysis above, I actually find this file in dae. Thank you! Cheers, Dave -- Read this topic online here: http://forum.openscenegraph.org/viewtopic.php?p=67446#67446 ___ osg-users mailing list osg-users@lists.openscenegraph.org http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org
Re: [osg-users] [build] DAE build failure.. dae.h not found
Hi Dave, I apologize for not providing the detail. Let me attempt to. Rather than compile the DOM, I downloaded the full set of 3rd party dependencies from this page. http://www.openscenegraph.org/index.php/download-section/dependencies The specific download I used is this one (http://download.osgvisual.org/3rdParty_VS2013_v120_x86_x64_V9_full.7z). There is a 1.4 and a 1.5 DOM within this set. I extracted the dependcies and placed them under my OSG source tree. You can see their location within my tree here. https://drive.google.com/open?id=0BzUf-8Ad-iIkQ0Jzd245eHZGTWM I went through several iterations of trying to get COLLADA configured. This is my current latest. https://drive.google.com/open?id=0BzUf-8Ad-iIkOGVWTDJ6LWNDT1E I believe COLLADA_INCLUDE_DIR shouldn't point to DOM headers. You should probably set it to c:/osg/OpenSceneGraph-3.4.0/OpenSceneGraph-3.4.0/3rdParty/x64/include . You should also set DEBUG variants of the boost libraries to point to libboost_*mt-gd*.lib (mt = multi threaded, gd = debug) and COLLADA_PCRECPP_LIBRARY_DEBUG to pcrecppd.lib (pcrecpp(with d suffix).lib) and also COLLADA_PCRE_LIBRARY_DEBUG to pcred.lib. Cheers, Robert M. ___ osg-users mailing list osg-users@lists.openscenegraph.org http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org
Re: [osg-users] [build] DAE build failure.. dae.h not found
Hi, I apologize for not providing the detail. Let me attempt to. Rather than compile the DOM, I downloaded the full set of 3rd party dependencies from this page. http://www.openscenegraph.org/index.php/download-section/dependencies The specific download I used is this one (http://download.osgvisual.org/3rdParty_VS2013_v120_x86_x64_V9_full.7z). There is a 1.4 and a 1.5 DOM within this set. I extracted the dependcies and placed them under my OSG source tree. You can see their location within my tree here. https://drive.google.com/open?id=0BzUf-8Ad-iIkQ0Jzd245eHZGTWM I went through several iterations of trying to get COLLADA configured. This is my current latest. https://drive.google.com/open?id=0BzUf-8Ad-iIkOGVWTDJ6LWNDT1E I believe that this set of dependencies has the compiled DOM you refer to. It is referenced in my COLLADA configuration. Drilling down into my 3rd party libs, one can see that compiled library. https://drive.google.com/open?id=0BzUf-8Ad-iIkSk15YjR3bTNFbkE I have not set any environment variables. I have only attempted to configure its location through CMAKE. This seemed to work for all other plugins. Should I also set the environment variable as you suggest? Hopefully this detail provides you with some more insights to help me. I do appreciate your response.. Thanks for looking at this. Cheers, Dave -- Read this topic online here: http://forum.openscenegraph.org/viewtopic.php?p=67440#67440 ___ osg-users mailing list osg-users@lists.openscenegraph.org http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org
Re: [osg-users] MSVS2015 3rdparty build
memory_leak wrote: > > Tryed this evening and I got it work, with some minor problems: > > * Curl does not build. I pulled source from git, and I don't know if it is > due to change in Curl or something else, but your cmake file is configured to > include some sources which are not found in curl source. They are few > curl_ntlm.c, curl_ntlm_msgs.c, curl_sasl_gssapi.c, curl_sasl_ssapi.c and > http_negotiate_sspi. Anyway Curl comes with it's own cmake file which works > fine "out of the box", so I have opted to comment it out completely from your > top level cmake. > Ok, it is rarely a good idea to build against master. In the Readme file there are specfied which tagged version which each library is tested against. I have just merged a pull request to support the latest stable version of cUrl (7.49.1). Also, there is no need to comment things out in the top level cmake file. If you do not want to include a certain library, just leave the corresponding *_SOURCE_DIR variable empty and the build script will omit the library from the build. memory_leak wrote: > * Giflib failes due to missing strings.h included from "ported" getopt.h. I > have included dummy strings.h which just includes standard string.h. I do not get this problem on Visual Studio 2015 x86. Which platform are you building on (compiler&architecture)? memory_leak wrote: > > * Is liFtiff oficially promoted to liff? :-) > Apparently! :-) I have fixed this spelling error. Regards Björn -- Read this topic online here: http://forum.openscenegraph.org/viewtopic.php?p=67438#67438 ___ osg-users mailing list osg-users@lists.openscenegraph.org http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org
Re: [osg-users] freetype build support on Windows
I don't know if I missunderstand your original post somehow, but I did build freetype twice in last few days with no issues whatsoever. I am on windows 10 64-bit and I have built it with VS2015 Community Edition, both debug and release libs. If you use cmake file that comes with freetype from git, just comment out check for mingw and shared libs: #if (WIN32 AND NOT MINGW AND BUILD_SHARED_LIBS) # message(FATAL_ERROR "Building shared libraries on Windows needs MinGW") #endif () If you use Björn's cmake scripts, you have to do nothing. -- Read this topic online here: http://forum.openscenegraph.org/viewtopic.php?p=67437#67437 ___ osg-users mailing list osg-users@lists.openscenegraph.org http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org
Re: [osg-users] [build] DAE build failure.. dae.h not found
Hi Dave, I don't know how you've gone about getting the dae plugin to work so it's not possible for us to know what might have gone wrong along the way. Given the lack of details about your set up the best I can do is general statements. For instance you need to compile the COLLADA_DOM for your platform then get the OSG's build system to detect this, such as by setting the COLLADA_DIR env var prior to running CMake/CMakeSetup. If this run correctly then the OSG' dae plugin should then build without problems. Robert. On 6 June 2016 at 00:07, Dave Sargrad wrote: > Hi, > I almost figured out how to get dae built. At least its building now. I'm > seeing the following build error: > > >> -- Build started: Project: Plugins dae, Configuration: Debug x64 -- >> 190> Building Custom Rule >> C:/osg/OpenSceneGraph-3.4.0/OpenSceneGraph-3.4.0/src/osgPlugins/dae/CMakeLists.txt >> 190> CMake does not need to re-run because >> C:\osg\OpenSceneGraph-3.4.0\build64\src\osgPlugins\dae\CMakeFiles\generate.stamp >> is up-to-date. >> 190> daeReader.cpp >> 190>c:\osg\openscenegraph-3.4.0\openscenegraph-3.4.0\src\osgplugins\dae\daeReader.h(19): >> fatal error C1083: Cannot open include file: 'dae.h': No such file or >> directory >> 190> daeRAnimations.cpp >> 190>c:\osg\openscenegraph-3.4.0\openscenegraph-3.4.0\src\osgplugins\dae\daeReader.h(19): >> fatal error C1083: Cannot open include file: 'dae.h': No such file or >> directory >> 190> daeRGeometry.cpp >> 190>c:\osg\openscenegraph-3.4.0\openscenegraph-3.4.0\src\osgplugins\dae\daeReader.h(19): >> fatal error C1083: Cannot open include file: 'dae.h': No such file or >> directory >> 190> daeRMaterials.cpp >> 190>c:\osg\openscenegraph-3.4.0\openscenegraph-3.4.0\src\osgplugins\dae\daeReader.h(19): >> fatal error C1083: Cannot open include file: 'dae.h': No such file or >> directory >> 190> daeRSceneObjects.cpp >> 190>c:\osg\openscenegraph-3.4.0\openscenegraph-3.4.0\src\osgplugins\dae\daeReader.h(19): >> fatal error C1083: Cannot open include file: 'dae.h': No such file or >> directory >> 190> daeRSkinning.cpp >> 190>c:\osg\openscenegraph-3.4.0\openscenegraph-3.4.0\src\osgplugins\dae\daeReader.h(19): >> fatal error C1083: Cannot open include file: 'dae.h': No such file or >> directory >> 190> daeRTransforms.cpp >> 190>c:\osg\openscenegraph-3.4.0\openscenegraph-3.4.0\src\osgplugins\dae\daeReader.h(19): >> fatal error C1083: Cannot open include file: 'dae.h': No such file or >> directory >> 190> daeWAnimations.cpp >> 190>c:\osg\openscenegraph-3.4.0\openscenegraph-3.4.0\src\osgplugins\dae\daeWriter.h(49): >> fatal error C1083: Cannot open include file: 'dae.h': No such file or >> directory >> 190> daeWGeometry.cpp > > > > Can you please tell me what I might be missing? > > Thank you! > > Cheers, > Dave > > -- > Read this topic online here: > http://forum.openscenegraph.org/viewtopic.php?p=67430#67430 > > > > > > ___ > osg-users mailing list > osg-users@lists.openscenegraph.org > http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org ___ osg-users mailing list osg-users@lists.openscenegraph.org http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org
Re: [osg-users] freetype build support on Windows
Hi Stuart, I believe it should be possible to have the FindFreetype.cmake detect that PNG is used and create an additional libraries var or just add it into FREETYPE_LIBRARIES var. Have a look at how the other Free*.cmake modules handle extra dependencies that need to be linked in. If this is done right it should be possible to have the OSG's freetype plugin to work without any special manual settings to add the PNG dependency. Robert. On 5 June 2016 at 22:51, Stuart Mentzer wrote: > Hi Robert, > > OK, I am testing a patched FindFreetype.cmake with OSG now. If that works > I'll submit it to CMake and to OSG. If it is better to post it to the > osg-submissions list rather than here I can do that but it is then separate > from the context of this discussion. > > It doesn't seem practical for us to fix freetype but I'll file a suggestion > with them to reconsider this build approach. For now, with a wiki note on > refreshing the source, the only freetype improvement we can benefit from is > making osgPlugins/freetype/CMakeLists.txt able to add PNG_LIBRARY (or other > freetype optional libs?) to the target libraries if needed. I don't know > enough CMake yet to do that automatically and adding a variable to pass to > the cmake call seems like cruft. If OSG has no benefit from PNG support in > freetype then a note not to enable it is probably the way to go. > > Cheers, > Stuart > > > On 6/5/2016 7:41 AM, Robert Osfield wrote: >> >> HI Stuart, >> >> It sounds like taking the CMake FindFreetype.cmake modifying to work >> and then getting this checked over by the cmake community as being >> suitable for them to merge and then sending the final rev along to me >> to merge would enable us to roll out the improved support prior to the >> next CMake release. If the CMake release is made before we push out >> 3.6 then we wouldn't need to add it locally. >> >> With the freetype wiring to PNG+ZLIB, this sounds like the could >> improve things with their own source/build system. I don't know >> freetype well enough to know how easy it would be to fix things to >> make it easier to switch. This type of issue is why the OSG has >> plugins and NodeKits - the core libraries are kept with minimal >> dependencies, this way the dependency chain doesn't pollute anything >> more than it needs to. >> >> Robert. >> >> >> >> On 5 June 2016 at 02:35, Stuart Mentzer wrote: >>> >>> Hi Robert, >>> >>> I have asked the CMake community about updating their FindFreetype.cmake >>> to >>> support Windows debug library naming and I will follow up to try and get >>> that fixed in upcoming releases. I was pointed to how they do it >>> correctly >>> for zlib so I could make a variant of their FindFreetype.cmake for OSG to >>> use until their fix is released. This would retain their support for the >>> old >>> and new include structure. If you'd like me to submit that let me know. >>> >>> Wrt the PNG on/off issue, I now understand the approach they use. The >>> upshot >>> is that as long as you refresh the freetype source tree you are building >>> with from the original code before each build you can switch PNG support >>> on >>> or off in the cmake command with -DWITH_PNG=ON or OFF and without >>> manually >>> editing ftoption.h. (Same holds for ZLIB support.) The reason is that the >>> build goes in and modifies ftoption.h in the source tree (as well as >>> making >>> a copy in the build tree) and the modification only uncomments those >>> defines, so you can't build with PNG enabled and then PNG disabled >>> without >>> refreshing the source first. This is an unfortunate approach but that is >>> what we are stuck with. Most builders don't switch the PNG or ZLIB >>> support >>> on and off so this probably doesn't often trip people up. The best we can >>> probably do is add a note on an appropriate wiki page. I added this >>> refresh >>> step to my build scripts. >>> >>> Stuart >>> >>> On 6/4/2016 3:36 PM, Robert Osfield wrote: >>> >>> Hi Stuart, >>> >>> It sounds like the version of Freestyle is broken or it requires a tweak >>> to >>> configuration. Have you approached the freetype community about these >>> issues. >>> >>> The debug vs release issue is something that would be worth raising with >>> the >>> cake community as it sounds like a revision to their Findfreetype.cmake. >>> >>> Robert >>> >>> On 3 Jun 2016 11:24 p.m., "Stuart Mentzer" wrote: Hi Robert, Here's what I found doing release and debug builds from yersterday's git master code with Visual C++ 2015: freetype even using -DWITH_PNG=OFF will still try to include png.h and for some reason requires ftoption.h (both copies) to be modified (or overridden) to comment out the line: #define FT_CONFIG_OPTION_USE_PNG This is unfortunate and actually makes it easier to build freetype with PNG support. With the freetype mods OSG builds including the freetype plugin. Configuring freetype with or without PNG s
Re: [osg-users] osg::View NO_LIGHT bug?
HI Nickolai, If you don't want the default global light then simple do: viewer.setLightingMode(osg::Viewer::NO_LIGHT); Rather than switching it to SKY_LIGHT! Robert. On 5 June 2016 at 22:44, Nickolai Medvedev wrote: > Well, we can deceive viewer if we make so: > > > Code: > viewer.getLight()->setAmbient(osg::Vec4(0.0,0.0,0.0,1.0)); > viewer.getLight()->setPosition(osg::Vec4(0.0,0.0,0.0,1.0)); > viewer.getLight()->setDiffuse(osg::Vec4(0.0,0.0,0.0,1.0)); > viewer.getLight()->setConstantAttenuation(0.003); > viewer.getLight()->setLinearAttenuation(0.1); > viewer.getLight()->setQuadraticAttenuation(0.1); > viewer.getLight()->setName("global_light"); > viewer.getLight()->setLightNum(0); > > viewer.setLightingMode(osg::View::SKY_LIGHT); > > > > The light source still be exist, but will be invisible. > > Cheers, > Nickolai > > -- > Read this topic online here: > http://forum.openscenegraph.org/viewtopic.php?p=67426#67426 > > > > > > ___ > osg-users mailing list > osg-users@lists.openscenegraph.org > http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org ___ osg-users mailing list osg-users@lists.openscenegraph.org http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org
Re: [osg-users] Preparing to make 3.5.3 dev release, please test
Hi Bjorn, Could you post a log of the warnings that this CMakeLists.txt works around? If there aren't too many we could just fix them. Robert. On 5 June 2016 at 21:27, Björn Blissing wrote: > The included CMakeLists file is for the 3ds plugin. > > My suggestion is to disable the variable shadowing warnings for this project, > since it is basically an external library although included as source files. > The other option is to fix all the variable name reuse for that project, > which feels kind of unnecessary as lib3ds is pretty much abandoned as project > (no activity since 2013). Autodesk has also stopped their development of this > format years ago. > > Regards > Björn > > > Från: osg-users för Robert > Osfield > Skickat: den 4 juni 2016 19:42:10 > Till: OpenSceneGraph Users > Ämne: Re: [osg-users] Preparing to make 3.5.3 dev release, please test > > Hi François, > > I have removed the register keyword usage from > Matrix_implementation.cpp and added the "" to the CMakeLists.txt. > These changes are now checked into git mater. > > Robert. > > On 4 June 2016 at 16:23, François Bérard wrote: >> Hi Robert, >> >> On 3/6/16 18:46, Robert Osfield wrote: >>> >>> Rather than add this to the root CMakeLists.txt file I have added a >>> Clang specific section to the src/osgPlugins/cfg/CMakeLists.txt thus: >>> >>> # lex/yacc generated files use register that causes warnings with >>> CLang under OSX so disable this warnings. >>> IF(${CMAKE_CXX_COMPILER_ID} STREQUAL "Clang") >>> SET(CMAKE_CXX_FLAGS ${CMAKE_CXX_FLAGS} -Wno-deprecated-register) >>> ENDIF() >>> >>> This is now checked into master. Could remove you your own mds and test >>> this? >> >> >> there is a small pb which breaks the build: >> >> Scanning dependencies of target osgdb_cfg >> [ 79%] Building CXX object >> src/osgPlugins/cfg/CMakeFiles/osgdb_cfg.dir/CameraConfig.cpp.o >> clang: error: no input files >> /bin/sh: -Wno-deprecated-register: command not found >> make[2]: *** >> [src/osgPlugins/cfg/CMakeFiles/osgdb_cfg.dir/CameraConfig.cpp.o] Error 127 >> make[1]: *** [src/osgPlugins/cfg/CMakeFiles/osgdb_cfg.dir/all] Error 2 >> make: *** [all] Error 2 >> >> >> adding double quotes fixes it: >> >> SET(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-deprecated-register") >> >> Also, the warning appears in osg/Matrix_implementation.cpp (see attached >> log), you may want to add the definition to the CMakelist of libosg. I tried >> by adding the IF block juste before the SETUP_LIBRARY, it worked. But >> removing the two register keywords in the matrix code may be the best >> approach: they are most probably always ignored by the compilers. >> >> ___ >> osg-users mailing list >> osg-users@lists.openscenegraph.org >> http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org >> > ___ > osg-users mailing list > osg-users@lists.openscenegraph.org > http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org > > ___ > osg-users mailing list > osg-users@lists.openscenegraph.org > http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org > ___ osg-users mailing list osg-users@lists.openscenegraph.org http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org