[darwinbuild] About the readlink redefinition in darwintrace.c

[Axel Luttgens]

On 15/03/06 19:58, Kevin Van Vechten wrote:

>
> On Mar 15, 2006, at 6:03 AM, Axel Luttgens wrote:
>
>> So, my question is: shouldn't line 219 of darwintrace.c read
>>
>>    ssize_t  readlink(const char * path, char * buf, ssize_t bufsiz) {
>>
>> instead of
>>
>>    ssize_t  readlink(const char * path, char * buf, size_t bufsiz) {
>
>
> The real issue is that the declaration of readlink in unistd.h  
> changed in tiger.  The darwintrace.c code is correct for Tiger, but  
> incorrect for earlier releases.


Oops! Sorry, I searched the list for possible items related to the 
above, but didn't think to have a look at Tiger's headers (which I 
nevertheless have here, on my disk; too lazy?).
Now, that change is rather unsuspected; does is mean that very, very 
long pathnames are going to be supported?

> We'll need to support both versions, conditionalized by one of the  
> macros in AvailabilityMacros.h.


Thanks a lot for your reply,
Axel