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 Jun 9, 2022
1 parent a3f9253 commit d2a1788
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 @@ -99,9 +99,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 @@ -115,6 +112,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 d2a1788

Please sign in to comment.