[FFmpeg-devel] [PATCH] configure: Check for the header sys/mmap.h when checking for mmap

[徐福隆]

There maybe a spelling mistake, which in "check_func_headers sys/mman.h mmap".
It looks like should be "sys/mmap.h"


Thanks
------------------ Original ------------------
From:                                                                                                                        "FFmpeg development discussions and patches"                                                                                    <martin at martin.st>;
Date: Thu, Jan 11, 2024 06:33 PM
To: "ffmpeg-devel"<ffmpeg-devel at ffmpeg.org>;

Subject: [FFmpeg-devel] [PATCH] configure: Check for the header sys/mmap.h when checking for mmap



Whenever mmap is used, we include sys/mmap.h unconditionally
within HAVE_MMAP ifdefs.

When building for a system (e.g. mingw) that normally lacks the
mmap function, but if building with profiling enabled, the
profiling runtime which gets linked in in each test executable,
can contain a function named mmap, causing stray detection of it.

This fixes an issue reported at
https://github.com/mstorsjo/llvm-mingw/issues/390.
---
 configure | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/configure b/configure
index 0b5e83bd20..99fa5260a4 100755
--- a/configure
+++ b/configure
@@ -6457,7 +6457,7 @@ check_func  getrusage
 check_func  gettimeofday
 check_func  isatty
 check_func  mkstemp
-check_func  mmap
+check_func_headers sys/mman.h mmap
 check_func  mprotect
 # Solaris has nanosleep in -lrt, OpenSolaris no longer needs that
 check_func_headers time.h nanosleep || check_lib nanosleep time.h nanosleep -lrt
-- 
2.39.3 (Apple Git-145)

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel at ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".