[Buildroot] [PATCH 1/1] package/attr: add --enable-multilib to fix build error for BL808 based boards

[Leo]

Hi,

I'll change my name, thanks for letting me know

The failure is on none of the auto builders. I have uploaded the config.log here: https://pastebin.com/ZzP6pNvx

And to fix this issue I added the --enable-multilib option which resolves the problem.

Regards,
Leonard Widler

Am 26. November 2023 22:56:44 MEZ schrieb "Yann E. MORIN" <yann.morin.1998 at free.fr>:
>leow149, All,
>
>Thank you for your contribution! Here is a quick review.
>
>First, we'll need you to use your real name as the author of the patch;
>we do not accept annonymous (od pseduonymou) contributions.
>
>On 2023-11-26 22:43 +0100, leow149 spake thusly:
>> This commit fixes a build error which specifically happens for Bouffallo Labs BL808 based boards, but might also happen on other RiscV64 Boards.
>
>Please wrap your commit log to ~72 chars.
>
>Waht is the error that you see? Could you see if there is a similar
>build failure on out autobulders:
>    http://autobuild.buildroot.org/?static=1&reason=attr-%
>
>The last build error for attr was more than a year ago now, so it is
>relatively surprising to see that attr does not build. So we'll need a
>bit more context.
>
>> Signed-off-by: leow149 <info.leo at widler.de>
>> ---
>>  package/attr/attr.mk | 5 +++--
>>  1 file changed, 3 insertions(+), 2 deletions(-)
>> 
>> diff --git a/package/attr/attr.mk b/package/attr/attr.mk
>> index 0327497e1e..89f7d21f0c 100644
>> --- a/package/attr/attr.mk
>> +++ b/package/attr/attr.mk
>> @@ -13,7 +13,8 @@ ATTR_CPE_ID_VENDOR = attr_project
>>  
>>  ATTR_INSTALL_STAGING = YES
>>  
>> -ATTR_CONF_OPTS = --disable-nls
>> -
>> +ATTR_CONF_OPTS = \
>> +	--disable-nls \
>> +	--enable-multilib
>
>Buildroot does not generate multilib toolchains, and when using an
>external toolchain that is multilib, the proper multilib diretory is
>copies over to the sysroot.
>
>So, --eanble-multilib looksa like the wrong approach, at least without
>more explanations.
>
>Can you elaborate on the issue you encounter, and what makes you think
>--enable-multilib is the correct solution?
>
>Thanks!
>
>Regards,
>Yann E. MORIN.
>
>>  $(eval $(autotools-package))
>>  $(eval $(host-autotools-package))
>> -- 
>> 2.30.2
>> 
>> _______________________________________________
>> buildroot mailing list
>> buildroot at buildroot.org
>> https://lists.buildroot.org/mailman/listinfo/buildroot
>
>-- 
>.-----------------.--------------------.------------------.--------------------.
>|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
>| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
>| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
>| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
>'------------------------------^-------^------------------^--------------------'
>_______________________________________________
>buildroot mailing list
>buildroot at buildroot.org
>https://lists.buildroot.org/mailman/listinfo/buildroot
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.buildroot.org/pipermail/buildroot/attachments/20231126/06717257/attachment-0001.html>