[GRASS-dev] Re: [GRASS GIS] #308: Compiler error while building python swig

[GRASS GIS]

#308: Compiler error while building python swig
----------------------------------+-----------------------------------------
  Reporter:  cgsbob               |       Owner:  grass-dev at lists.osgeo.org
      Type:  defect               |      Status:  new                      
  Priority:  major                |   Milestone:  6.4.0                    
 Component:  SWIG (all bindings)  |     Version:  svn-develbranch6         
Resolution:                       |    Keywords:  python swig              
  Platform:  Unspecified          |         Cpu:  Unspecified              
----------------------------------+-----------------------------------------
Comment (by hamish):

 I checked in some fixes to devbr6 SVN but it still doesn't build.

 {{{
 python_grass6_wrap.c: In function '_wrap_Cluster_nbands_set':
 python_grass6_wrap.c:48044: error: dereferencing pointer to incomplete
 type
 python_grass6_wrap.c: In function '_wrap_Cluster_nbands_get':
 python_grass6_wrap.c:48067: error: dereferencing pointer to incomplete
 type
 python_grass6_wrap.c: In function '_wrap_Cluster_npoints_set':
 python_grass6_wrap.c:48097: error: dereferencing pointer to incomplete
 type
 python_grass6_wrap.c: In function '_wrap_Cluster_npoints_get':
 ...
 python_grass6_wrap.c: In function '_wrap_new_Cluster':
 python_grass6_wrap.c:49087: error: invalid application of 'sizeof' to
 incomplete type 'struct Cluster'
 ...
 python_grass6_wrap.c: At top level:
 python_grass6_wrap.c:82965: warning: function declaration isn't a
 prototype
 make: *** [python_grass6_wrap.o] Error 1
 }}}




 ----

 Auto-generation of gis.i seems to be an important thing, so wrt to
 tackling that next:

  - gis.i seems to be based on gis.h circa 12/17/04 (r16393)

 After removing comments and trailing whitespace, then diff'ing versus
 gis.h r16393 the changes between gis.h and gis.i are as follows:

 {{{
 -extern CELL CELL_NODATA;
 +/*extern CELL CELL_NODATA; Sajith */


 @@ -159,10 +101,17 @@
         unsigned char red;
         unsigned char grn;
         unsigned char blu;
 -    } low, high;
 +    }  high;
 +    struct
 +    {
 +       DCELL value;
 +       unsigned char red;
 +       unsigned char grn;
 +       unsigned char blu;
 +    }  low;

 -    struct _Color_Rule_ *next;
 -    struct _Color_Rule_ *prev;
 +/*    struct _Color_Rule_ *next;
 +    struct _Color_Rule_ *prev;  Commented By sajith I am confued
 here....*/
  };

  struct _Color_Info_
 }}}


 The duplication of the high,low structs was done by Sajith AFAICT.

 I've no idea why of course, but that's what it is.


 Hamish

-- 
Ticket URL: <http://trac.osgeo.org/grass/ticket/308#comment:11>
GRASS GIS <http://grass.osgeo.org>