Re: Build on Solaris

DINH Viêt Hoà <[email protected]>
Newsgroups gmane.mail.libetpan.user
Message-ID <[email protected]>
On 26 Jul 2005, at 02:24, Alex S Moore wrote:

> DINH Viet Hoa wrote:
>
>> On 23 Jul 2005, at 19:06, Alex S Moore wrote:
>>
>>> ---
>>>  cc -c -I../../include -I../.. -I. -fast -xarch=v8 -I/opt/csw/ 
>>> bdb4/ include -I/opt/csw/include -D_REENTRANT -DHAVE_CONFIG_H   
>>> mailstream.c  -KPIC -DPIC -o .libs/mailstream.o
>>> "mailstream.c", line 402: syntax error before or at: .
>>> cc: acomp failed for mailstream.c
>>> *** Error code 1
>>> make: Fatal error: Command failed for target `mailstream.lo'
>>> Current working directory /export/medusa/asmoore/build/  
>>> libetpan-0.38/src/data-types
>>> *** Error code 1
>>> ---
>>>
>> You will see much problems like this. They will be surrounded by  
>> such  defines.
>> #ifdef _MSC_VER
>> #else
>> #endif
>> Just replace "#ifdef _MSC_VER" with "#if 1".
>> You may try to use gcc to compile.
>> However, I'll try to find write some fix for Sun Compiler.
>> Whenever there is constant structures defined this way, you may  
>> use  _MSC_VER alternative.
>> And tell about the result.
>>
>
> The compiler complained about "#if 1" not being an identifier.   
> Setting
> _MSC_VER = 1 did not work at all.  Mainly from problems with includes.
> I am on Solaris.
>
> What I did was patch the source to use a new defines where needed.   
> The
> patch file is attached.  I manually added the defines to config.h  
> and am
> not sure of the changes required to make the new defines end up
> in config.h.  I have not been able to test the new library, but it
> compiled just fine.  I will be testing the library over the next  
> week or
> so with the sylpheed-claws application.  But my testing probably will
> not hit all of the combinations of tests needed.
>
> There were a couple of other files that did not have _MSC_VER defined,
> but I had to add it.  I add the defines for _MSC_VER to these  
> files. Maybe the code is already in cvs.
>
> Can you add this patchlist to your source and add the code needed
> for the __SUNPRO_CC defines?

Thanks for the patch, it will be scheduled for commit.

-- 
DINH Viet Hoa



-------------------------------------------------------
SF.Net email is sponsored by: Discover Easy Linux Migration Strategies
from IBM. Find simple to follow Roadmaps, straightforward articles,
informative Webcasts and more! Get everything you need to get up to
speed, fast. http://ads.osdn.com/?ad_id=7477&alloc_id=16492&op=click
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.