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 Aug 9, 2022
1 parent 52dcca4 commit 8598071
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 8598071

Please sign in to comment.