linphone-users
[Top][All Lists]
Advanced

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

[Linphone-users] using __linux__ instead of __linux


From: Daniel Pocock
Subject: [Linphone-users] using __linux__ instead of __linux
Date: Thu, 1 Jul 2021 01:15:44 +0200
User-agent: Mozilla/5.0 (X11; Linux ppc64le; rv:68.0) Gecko/20100101 Thunderbird/68.12.0


I found __linux in Linphone.  It is not completely portable on all
architectures, here is a similar bug in libusb:

https://github.com/libusb/libusb/pull/315



I had to patch it here to make it compile, I can't submit the patch
because it looks like Gitlab doesn't allow developers to create new
accounts.  If somebody can help me with a Gitlab account I'll be happy
to submit the merge request for this.

https://gitlab.linphone.org/BC/public/liblinphone


It is necessary to replace __linux with __linux__ in each of these files:


$ find . -type f -name '*.c*' -exec grep -H __linux {} \;

./coreapi/linphonecore.c:#ifdef __linux

./src/core/paths/paths.cpp:#elif defined(__linux)

./tester/liblinphone_tester.c:#ifdef __linux

./tester/message_tester.c:#if defined(__linux__) && !defined(__ANDROID__)

./tester/register_tester.c:#ifdef __linux

./tester/setup_tester.c:#elif defined(__linux) || (defined(__APPLE__) &&
!TARGET_OS_IPHONE) || defined(_WIN32)

./obj-powerpc64le-linux-gnu/CMakeFiles/3.16.3/CompilerIdC/CMakeCCompilerId.c:#if
defined(__linux) || defined(__linux__) || defined(linux)

./obj-powerpc64le-linux-gnu/CMakeFiles/3.16.3/CompilerIdCXX/CMakeCXXCompilerId.cpp:#if
defined(__linux) || defined(__linux__) || defined(linux)



reply via email to

[Prev in Thread] Current Thread [Next in Thread]