Re: [Nebula-Discuss] Linux compilation

[Vadim M. <va...@cu...>]

Marco Antonio Gómez Martín wrote:
>    Hello!
> 
>    I have been compiling nebula2tools target (revision 3218) in 
> GNU/Linux (Ubuntu, kernel 2.6.17). I have detected some errors in the 
> current SVN state, that I  have changed locally. Just in case main 
> developers want to commit them to SVN, I am going to detail what I have 
> found (and changed), attaching the associated patch files. I also 
> include some explanations about the changes to ease the task of 
> considering whether commit them or not.

Thanks for doing all this, I've made some notes below as to what I 
actually ended up doing.

> SVN IGNORE
> ----------
> 
> Please add the svn:ignore properties to nebula2/bin/linux directory.

Done.

> SSE SUPPORT
> -----------
> 
> Nowadays, g++ is not invoked using the correct flags to support MMX and 
> SSE extensions. I have changed nebula2/buildsys3/config.mak to do that. 
> Unfortunately, SSE code is Win32 specific, so it do not compile in 
> Linux. With this patch, source code use the __USE_SSE macro to determine 
> whether the SSE code must be compiled or not.

I left out the change to config.mak, seeing as SSE doesn't build anyway. 
The .h don't need to be modified since they don't get included outside 
the corresponding .cc unless the existing __USE_SSE__ flag is defined 
(no need for another __USE_SSE flag).

> NETWORK
> -------
> 
> I do not know in detail network modules and targets. I have learnt
> that some of the modules make use of windows specific functions and
> defines, such as nbuddyclient, that uses WinBase.h.

I managed to get the network stuff building, the win32 specific stuff 
was a minor issue (console text color setting and winsock initialization 
that I just #ifdef-ed out). The majority of the work involved fixing the 
mixed case includes (someone got a bit sloppy). So, it builds, but I 
leave it up to you to figure out if it actually works :)

> WIN32 SPECIFICS CODE
> --------------------
> 
> Added #define's to code created for Win32 platform.

Instead of #ifdef-ing the win32 specific code I fixed the .bld files.

> WIN32 SPECIFICS MODULES AND TARGETS
> -----------------------------------
> 
> - audio3.bld: direct sound modules. I have seen that naudiofile is 
> windows dependent, because it make use of WAVEFORMATEX type. That forced 
> me to remove this module from Linux compilation.

If someone wants to fix nAudioFile that shouldn't be too hard.

> WARNINGS
> --------
> 
> G++ generates a great amount of warnings. I have just resolve two of them.

I left this patch out for now, I might commit it later. It probably 
would be nice if we could get rid of all those warnings though.


-+ enlight +-