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 Nov 29, 2022
1 parent 4d08e2d commit ee1d98d
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 ee1d98d

Please sign in to comment.