<div dir="ltr"><div>This part of the patch still sounds weird to me:<br><br>ifneq (,$(filter mips64%el mips64%el,$(DEB_HOST_ARCH)))<br><br></div>The repetition of "mips64%el" looks unnecessary to me.<br><div><div><br></div><div>Besides, since the aim is to catch all future mips64* architectures, shouldn't  this be enough:<br><br>ifneq (,$(filter mips64%,$(DEB_HOST_ARCH)))<br></div><div><br></div><div>Unless I am missing something here. Let me know what you think.<br></div><div><br></div><div>Ghis<br><br></div><div><div><div><div class="gmail_extra"><br><div class="gmail_quote">2015-11-30 8:00 GMT+00:00 YunQiang Su <span dir="ltr"><<a href="mailto:wzssyqa@gmail.com" target="_blank">wzssyqa@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">I tested this patch. It works well for mips64el.<br>
<div class=""><div class="h5"><br>
On Fri, Nov 27, 2015 at 11:40 AM, YunQiang Su <<a href="mailto:wzssyqa@gmail.com">wzssyqa@gmail.com</a>> wrote:<br>
> On Wed, Nov 25, 2015 at 6:23 PM, Ghislain Vaillant <<a href="mailto:ghisvail@gmail.com">ghisvail@gmail.com</a>> wrote:<br>
>> This is the relevant portion of the proposed patch:<br>
>><br>
>>>  ifeq ($(DEB_HOST_ARCH),ppc64el)<br>
>>>  DEB_CFLAGS_MAINT_APPEND += -mno-altivec<br>
>>> +DEB_CXXFLAGS_MAINT_APPEND += -mno-altivec<br>
>>> +endif<br>
>>> +# Build without Altivec to prevent FTBFS on ppc64el.<br>
>>> +ifneq (,$(filter mips64%el mips64%el,$(DEB_HOST_ARCH)))<br>
>>> +DEB_CFLAGS_MAINT_APPEND += -mxgot<br>
>>> +DEB_CXXFLAGS_MAINT_APPEND += -mxgot<br>
>>>  endif<br>
>><br>
>> But the explanation is:<br>
>><br>
>>> filter mips%64 mips%64el,$(DEB_HOST_ARCH)<br>
>>> is due to we may has some other mips64 architectures in future,<br>
>>> for example: mips64r6{el}.<br>
>><br>
>> The filter string differs between the patch and explanation:<br>
><br>
> The patch is correct.<br>
><br>
> Sorry for it.<br>
><br>
>><br>
>> filter mips64%el mips64%el<br>
>><br>
>> and<br>
>><br>
>> filter mips%64 mips%64el.<br>
>><br>
>> One is twice the same (typo?), the other has the percent sign placed<br>
>> differently.<br>
>><br>
>> Please confirm the correct filter command, otherwise I cannot apply this<br>
>> patch.<br>
>><br>
>> Thanks,<br>
>> Ghis<br>
><br>
><br>
><br>
> --<br>
> YunQiang Su<br>
<br>
<br>
<br>
</div></div><span class=""><font color="#888888">--<br>
YunQiang Su<br>
</font></span></blockquote></div><br></div></div></div></div></div></div>