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 Mar 24, 2023
1 parent 08acdd1 commit 81d8a16
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 @@ -111,9 +111,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 @@ -127,6 +124,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 81d8a16

Please sign in to comment.