Hi Vincent,

Thanks. I'll push it in the following days.

Kind Regards,
Nemanja Lukic

-----Original Message-----
From: Vicente Olivert Riera [mailto:[email protected]] 
Sent: Friday, September 12, 2014 4:19 PM
To: Nemanja Lukic
Cc: [email protected]
Subject: Re: pixman-0.32.6 fails to build on mips32r2

Hello Nemanja,

I have attached the new patch to the bug report:

https://bugs.freedesktop.org/show_bug.cgi?id=83358#c1

Could you please commit it upstream?

Thank you.

On 09/11/2014 11:02 AM, Vicente Olivert Riera wrote:
> On 09/11/2014 10:57 AM, Nemanja Lukic wrote:
>> Hi Vincent,
>>
>> Patch I sent to you was not intended for pushing upstream :) It was 
>> just for check.
>
> Ah, ok, I see :P Sorry about that then.
>
>> Your patch LGTM, and it should be pushed on Pixman trunk.
>
> Thanks! Could you please let me know when you push it? And also, could 
> you please update the bug report and close it as resolved/fixed?
>
>> Thanks,
>
> Thanks to you Nemanja :)
>
>> Nemanja Lukic
>>
>> -----Original Message-----
>> From: Vicente Olivert Riera [mailto:[email protected]]
>> Sent: Thursday, September 11, 2014 11:34 AM
>> To: Nemanja Lukic
>> Cc: [email protected]
>> Subject: Re: pixman-0.32.6 fails to build on mips32r2
>>
>> Hello Nemanja,
>>
>> yes, it works. But your patch wrong because it can't be applied 
>> without reverting your last commit which was already pushed upstream. 
>> So, the right solution would be to apply a new patch which removes 
>> the wrong lines and add the right lines. I have attached a valid 
>> patch to this email. Please, check it.
>>
>> Thanks.
>>
>> On 09/11/2014 10:02 AM, Nemanja Lukic wrote:
>>> Hi Vincent,
>>>
>>> Please take a look at attached fix.
>>> This one should solve the problem, without duplicating LEAF_MIPS32R2
>> macro.
>>> Trick is to use pre-processor on the right level.
>>>
>>> Please check on your side,
>>> Nemanja Lukic
>>>
>>> -----Original Message-----
>>> From: Vicente Olivert Riera [mailto:[email protected]]
>>> Sent: Thursday, September 4, 2014 1:51 PM
>>> To: [email protected]
>>> Cc: [email protected]
>>> Subject: pixman-0.32.6 fails to build on mips32r2
>>>
>>> Could you please have a look to this bug report?
>>>
>>> https://bugs.freedesktop.org/show_bug.cgi?id=83358
>>>
>>> Thank you.
>>>
>>
>>
>> --
>> Vincent
>>
>
>


--
Vincent

_______________________________________________
Pixman mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/pixman

Reply via email to