Skip to content

Commit cbb90c7

Browse files
committed
ext/fileinfo/config.{m4,w32}: don't compile strcasestr.c
Unlike the upstream library, PHP's copy of libmagic doesn't use the strcasestr() function, so there is no reason to include strcasestr.c in the list of source files.
1 parent 800800f commit cbb90c7

2 files changed

Lines changed: 1 addition & 6 deletions

File tree

ext/fileinfo/config.m4

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -29,11 +29,6 @@ if test "$PHP_FILEINFO" != "no"; then
2929

3030
AC_CHECK_HEADERS([sys/sysmacros.h])
3131

32-
AC_CHECK_FUNCS([strcasestr],,[
33-
AC_MSG_NOTICE([using libmagic strcasestr implementation])
34-
libmagic_sources="$libmagic_sources libmagic/strcasestr.c"
35-
])
36-
3732
AX_GCC_FUNC_ATTRIBUTE([visibility])
3833

3934
PHP_NEW_EXTENSION([fileinfo],

ext/fileinfo/config.w32

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ if (PHP_FILEINFO != 'no') {
88
encoding.c fsmagic.c funcs.c \
99
is_json.c is_tar.c is_simh.c magic.c print.c \
1010
readcdf.c softmagic.c der.c \
11-
strcasestr.c buffer.c is_csv.c";
11+
buffer.c is_csv.c";
1212

1313
EXTENSION('fileinfo', 'fileinfo.c php_libmagic.c', true, "/I" + configure_module_dirname + "/libmagic /I" + configure_module_dirname);
1414
ADD_EXTENSION_DEP('fileinfo', 'pcre');

0 commit comments

Comments
 (0)