Complete.Org: Mailing Lists: Archives: freeciv-dev: October 2002:
[Freeciv-Dev] Re: Problem Compiling Client Under OpenVMS With DEC C.
Home

[Freeciv-Dev] Re: Problem Compiling Client Under OpenVMS With DEC C.

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: Davide Pagnin <nightmare@xxxxxxxxxx>
Cc: Freeciv Developers ML <freeciv-dev@xxxxxxxxxxx>, Robert Alan Byer <byer@xxxxxxxxxxxxxxxxxxx>
Subject: [Freeciv-Dev] Re: Problem Compiling Client Under OpenVMS With DEC C.
From: Mike Kaufman <kaufman@xxxxxxxxxxxxxxxxxxxxxx>
Date: Tue, 8 Oct 2002 13:36:31 -0500

Robert-

thanks for the testing. this should be fixed for 1.14.0.

I agree with the suggestion. Anybody have any problems with removing
common/attribute.h? Davide, would you make the patch?

-mike

On Tue, Oct 08, 2002 at 07:31:06PM +0200, Davide Pagnin wrote:
> On Tue, 2002-10-08 at 19:03, Mike Kaufman wrote:
> > wasn't the fact that we have two attribute.h's a problem for some of the
> > handheld developers? Is the compiler trying to find the wrong attribute.h
> > and choking when it sees fc__attribute()?
> 
> Yes, definitely you've catch the problem.
> This can explain why civserver compiles and instead civclient not.
> (it uses the wrong attribute.h file)
> 
> > 
> > I might suggest renaming attribute.h in the client to client_attribute.h
> > (with appropriate changes elsewhere) and see if that solves it. If you
> > check your preprocessor output, fc__attribute should not be appearing...
> 
> Isn't better to get rid of the common/attribute.h file and including it
> into common/shared.h, with the related necessary include rules removal?
> 
> (I mean, also in CVS tree as well...)
> 
> Inside common/attribute.h file there is only that little define that
> checks __GCC__ define, for how to substitute fc__attribute.
> 
> This is only my 2 cents.
> 
> > 
> > -mike
> 
>       Ciao, Davide
> 


[Prev in Thread] Current Thread [Next in Thread]