Re: [Xenomai] [PATCH 1/1] posix: fix build failure when HAVE_OPEN64 is undefined.

[Gilles Chanteperdrix <gilles.chanteperdrix@xenomai.org>]

On 12/29/2013 09:42 PM, Romain Naour wrote:
> Le 29/12/2013 18:56, Gilles Chanteperdrix a écrit :
>> On 12/29/2013 06:00 PM, Romain Naour wrote:
>>> fixes: http://autobuild.buildroot.org/results/d59/d590be34637a4043c929ce615f9db504484be435/build-end.log
>>>
>>> Signed-off-by: Romain Naour <romain.naour@openwide.fr>
>>> ---
>>>   src/skins/posix/rtdm.c | 3 ++-
>>>   1 file changed, 2 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/src/skins/posix/rtdm.c b/src/skins/posix/rtdm.c
>>> index 5840c64..d6c87dc 100644
>>> --- a/src/skins/posix/rtdm.c
>>> +++ b/src/skins/posix/rtdm.c
>>> @@ -107,8 +107,9 @@ int __wrap_open64(const char *path, int oflag, ...)
>>>   			errno = EMFILE;
>>>   			ret = -1;
>>>   		}
>>> +	}
>>>   #endif
>>> -	} else {
>>> +	else {
>>>   		errno = -ret;
>>>   		ret = -1;
>>>   	}
>>>
>> xenomai-2.6$ git apply ~/foo.eml
>> /home/gilles/foo.eml:79: trailing whitespace.
>>          }
>> /home/gilles/foo.eml:82: trailing whitespace.
>>          else {
>> error: patch failed: src/skins/posix/rtdm.c:107
>> error: src/skins/posix/rtdm.c: patch does not apply
>>
>> Are you working from xenomai-2.6 git?
>>
> I have no error while I do:
> git diff --check HEAD HEAD~1
> 
> But I'm able to reproduce this error when I save the email to foo.eml 
> with Thunderbird...
> 
> Have you whitespace errors with my previous patch too ?

git am accepts the patch, it seems git apply does not because of
whitespace issues. I believe you did not introduce the whitespace
issues, they were there before.

> 
> Best regards,
> Romain Naour
> 


-- 
                                                                Gilles.