apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Curt Arnold <carn...@apache.org>
Subject Re: [PATCH] Don't define POSIX types {p,g,u}id_t
Date Tue, 26 Apr 2005 23:31:42 GMT
I had to do some hacks around this to get APR to build with Borland C++  
(see  
http://cvs.apache.org/viewcvs.cgi/logging-log4cxx/apr-1.1.0.patch? 
rev=1.2&view=markup) which had uid_t and gid_t already defined.

BTW: Do I have any hope of getting the patches to support Borland and  
MinGW compilation reviewed and incorporated into the SVN?


On Apr 26, 2005, at 2:03 PM, William A. Rowe, Jr. wrote:

> At 01:27 PM 4/26/2005, Erik Huelsmann wrote:
>
>> I'm sorry to take my annoyance out on you, since it's not (only) you
>> who caused this (I'm getting an equally cooperative reaction with the
>> ruby team: "It's APR's fault: they should have used #define instead of
>> typedef.") But if you put it that way, there's hardly any reason to do
>> anything but major releases if you take the rules this litteral.
>
> I've suggested we change this to a #define on Win32.
>
> The reason is that it's simple enough to #undef it after including
> the header.
>
> Your comments are well taken; this will be addressed in the 2.0.0
> release (pid_t will go poof altogether.)
>
> Bill
>
>


Mime
View raw message