Skip to content

Commit

Permalink
Fix cmake to add big endian compile option
Browse files Browse the repository at this point in the history
Without this fix, mediastreamer2 doesn't have this compile option (CMakeLists.txt#L669) with CMake and it might create RTP packet with wrong payload type in header due to markbit field update (src/otherfilters/msrtp.c#L633)
  • Loading branch information
abioteau committed Sep 13, 2022
1 parent 1647471 commit 38d4d28
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -100,9 +100,6 @@ check_symbol_exists(sendmsg "sys/socket.h" HAVE_SENDMSG)

include(TestBigEndian)
test_big_endian(WORDS_BIGENDIAN)
if(WORDS_BIGENDIAN)
set(ORTP_BIGENDIAN 1)
endif()


include_directories(
Expand All @@ -116,6 +113,9 @@ endif()


set(ORTP_CPPFLAGS ${BCTOOLBOX_CPPFLAGS})
if(WORDS_BIGENDIAN)
list(APPEND ORTP_CPPFLAGS "-DORTP_BIGENDIAN")
endif()
if(ENABLE_STATIC)
list(APPEND ORTP_CPPFLAGS "-DORTP_STATIC")
endif()
Expand Down

0 comments on commit 38d4d28

Please sign in to comment.