gerwindehaan / osgswig

Python bindings for OpenSceneGraph (auto-export from code.google.com/p/osgswig)
Other
1 stars 1 forks source link

Typo in #if ressurrects the MixinVector problem #18

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
I couldn't get osgswig to build in my Mac environment (Leopard, 10.5.5). 
After some bumbling around, I came up with the following changes, which
make the build work.

Index: osg.i
===================================================================
--- osg.i   (revision 171)
+++ osg.i   (working copy)
@@ -70,7 +70,7 @@
 #include <osg/AnimationPath>
 #include <osg/ArgumentParser>

-#if (OPENSCENEGRAPH_SO_VERSION > 41)
+#if (OPENSCENEGRAPH_SOVERSION > 41)
 #include <osg/MixinVector>
 #endif

@@ -500,7 +500,7 @@
 %include osg/Referenced
 %include osg/ref_ptr

-#if (OPENSCENEGRAPH_SO_VERSION > 41)
+#if (OPENSCENEGRAPH_SOVERSION > 41)
 %include osg/MixinVector
 #endif

Original issue reported on code.google.com by randolph...@gmail.com on 24 Nov 2008 at 6:01

GoogleCodeExporter commented 9 years ago
Thanks, I will patch it tomorrow!

Original comment by hartmut on 24 Nov 2008 at 9:27

GoogleCodeExporter commented 9 years ago
Goes in fixing queue

Original comment by hartmut on 24 Nov 2008 at 9:29

GoogleCodeExporter commented 9 years ago
I have fixed this issue, however this has introduced new warnings such as 
    Nothing known about base class 'MixinVector< osg::Vec2f >'. Ignored.
    Maybe you forgot to instantiate 'MixinVector< osg::Vec2f >' using %template.

I tried to add the template for MixinVector, but it introduced new errors such 
as...
   osgPYTHON_wrap.cxx(127540) : error C2248: 'osg::MixinVector<ValueT>::vector_type' : cannot access 
private typedef declared in class 'osg::MixinVector<ValueT>'
        with
        [
            ValueT=osg::Vec3
        ]
        and
        [
            ValueT=osg::Vec3
        ]
...

Original comment by jky...@gmail.com on 7 Jan 2009 at 3:13

GoogleCodeExporter commented 9 years ago
I suppose these errors are solved by the MixinVector fixes, introduced almost 
over a year ago, closing issue

Original comment by gerwinde...@gmail.com on 21 Mar 2011 at 10:26