[igraph] Re: igraph 0.6 visual studio 2008 (vc9 sp1) bugfixes

[Laurence Muller]

Hi Tamas,


On May 20, 5:12 am, Tamas Nepusz <address@hidden> wrote:
> Hi Laurence,
>
> Thanks for your email, it's really helpful as we don't test igraph regularly 
> on MSVC.

No problem, I'm glad I can help you guys out with this!

> > (5.4) - community.c (error C2054: expected '(' to follow 'inline')
> > 'inline' does not work in *.c files when using visual studio, so
> > either change the extension to *.cpp or remove the 'inline' keyword.
>
> I used a slightly different solution here:
>
> #ifdef _MSC_VER
> #  define INLINE __forceinline
> #else
> #  define INLINE inline
> #endif
>
> INLINE igraph_real_t igraph_i_multilevel_community_modularity_gain(...)
>

ok

> > (8) - Building..
> > When trying to build you will end up with these errors:
> > [...]
> > The problem is that these functions are defined but not implemented?
> > extern int igraph_dl_yyparse(void);
> > extern FILE *igraph_dl_yyin;
> > extern int igraph_i_dl_mode;
>
> These functions are implemented in foreign-dl-lexer.c and 
> foreign-dl-parser.c, which are in turn generated by bison from 
> foreign-dl-lexer.l and foreign-dl-parser.y. The generation is done during the 
> build phase, so if you simply check out igraph, run ./configure and then run 
> make msvc, these files won't be generated and included in the ZIP. At least 
> that's my first guess. So, in your step 1, I would actually run "make" before 
> "make msvc". Note that you will need bison on your Ubuntu box. The final 
> igraph release will include these generated files, so you won't have the same 
> problems if you download the official released tarball for 0.6 (which we 
> don't have yet as 0.6 is not released).

I'm using my ubuntu machine (with all the extra required tools) to do
a checkout of the repository and to initially run the configure and
make commands :). I'll try to checkout your fixes soon!

Best regards,
- Laurence