[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#1054463: kodi: FTBFS: add support for loongarch64



Hi,

   Thanks for your reply.

On Tue, 24 Oct 2023 11:05:39 +0000 Vasyl Gello wrote:

> Hi Dandan,
>
> Please ensure you have arch-specific constants in include files fixed like here: https://github.com/xbmc/xbmc/commit/69611c8fa937800d1c5d7897e798d56d770d7d3c
>

> Your upstream PR touches only system info display.

The extra compiler ifdefs has been submitted in the previous pull-request.
Please see this link: https://github.com/xbmc/xbmc/pull/22062/files

Except for system info, other files needed to add architectural support already contain loongarch definitions in kodi 2:20.2+dfsg-4.
root@localhost:/home/kodi/kodi-20.2+dfsg# grep "loongarch" -ri . |grep -v .pc
./xbmc/utils/MathUtils.h:    defined(__loongarch__) || \
./xbmc/cores/DllLoader/ldt_keeper.c:    !defined(__loongarch__) && \
./xbmc/cores/DllLoader/DllLoader.h:    !defined(__loongarch__) && \
./cmake/scripts/linux/ArchSetup.cmake:  elseif(CPU MATCHES loongarch64)
./cmake/scripts/linux/ArchSetup.cmake:    set(ARCH loongarch64)

thanks,
Dandan Zhang



Reply to: