View Issue Details

IDProjectCategoryView StatusLast Update
0000163fileGeneralpublic2020-05-31 00:11
Reporterpombredanne Assigned Tochristos  
PrioritynormalSeveritymajorReproducibilityalways
Status feedbackResolutionopen 
Product Version5.38 
Summary0000163: Build fails on mingw64
DescriptionWhen trying to update the msys2.org package for file to 5.38, the build fails
See the PR at https://github.com/msys2/MINGW-packages/pull/6495 and in particular the build log at https://dev.azure.com/msys2/mingw/_build/results?buildId=5178&view=logs&jobId=d025390b-1dd2-5eea-69df-6ceb0b9eeced&j=d025390b-1dd2-5eea-69df-6ceb0b9eeced&t=7bf27dc4-4036-5dcd-5905-ed58d9f24e1f

I have also attached a local build log from running this in a Windows VM
TagsNo tags attached.

Activities

pombredanne

2020-05-19 13:48

reporter  

mingw-w64-file-5.38-1-x86_64-build.log (39,979 bytes)   
configure: loading site script /mingw64/etc/config.site
checking for a BSD-compatible install... /usr/bin/install -c
checking whether build environment is sane... yes
checking for a thread-safe mkdir -p... /usr/bin/mkdir -p
checking for gawk... gawk
checking whether make sets $(MAKE)... yes
checking whether make supports nested variables... yes
checking whether make supports nested variables... (cached) yes
checking for builtin ELF support... yes
checking for ELF core file support... yes
checking for zlib support... 
checking for bzlib support... 
checking for xzlib support... 
checking for libseccomp support... no
checking for file formats in man section 5... no
checking build system type... x86_64-w64-mingw32
checking host system type... x86_64-w64-mingw32
checking whether make supports the include directive... yes (GNU style)
checking for x86_64-w64-mingw32-gcc... x86_64-w64-mingw32-gcc
checking whether the C compiler works... yes
checking for C compiler default output file name... a.exe
checking for suffix of executables... .exe
checking whether we are cross compiling... no
checking for suffix of object files... o
checking whether we are using the GNU C compiler... yes
checking whether x86_64-w64-mingw32-gcc accepts -g... yes
checking for x86_64-w64-mingw32-gcc option to accept ISO C89... none needed
checking whether x86_64-w64-mingw32-gcc understands -c and -o together... yes
checking dependency style of x86_64-w64-mingw32-gcc... gcc3
checking for x86_64-w64-mingw32-gcc option to accept ISO C99... none needed
checking for x86_64-w64-mingw32-gcc option to accept ISO Standard C... (cached) none needed
checking how to run the C preprocessor... x86_64-w64-mingw32-gcc -E
checking for grep that handles long lines and -e... /usr/bin/grep
checking for egrep... /usr/bin/grep -E
checking for ANSI C header files... yes
checking for sys/types.h... yes
checking for sys/stat.h... yes
checking for stdlib.h... yes
checking for string.h... yes
checking for memory.h... yes
checking for strings.h... yes
checking for inttypes.h... yes
checking for stdint.h... yes
checking for unistd.h... yes
checking minix/config.h usability... no
checking minix/config.h presence... no
checking for minix/config.h... no
checking whether it is safe to define __EXTENSIONS__... yes
checking whether byte ordering is bigendian... no
checking whether ln -s works... no, using cp -pR
checking how to print strings... printf
checking for a sed that does not truncate output... /usr/bin/sed
checking for fgrep... /usr/bin/grep -F
checking for ld used by x86_64-w64-mingw32-gcc... C:/msys/mingw64/x86_64-w64-mingw32/bin/ld.exe
checking if the linker (C:/msys/mingw64/x86_64-w64-mingw32/bin/ld.exe) is GNU ld... yes
checking for BSD- or MS-compatible name lister (nm)... /mingw64/bin/nm -B
checking the name lister (/mingw64/bin/nm -B) interface... BSD nm
checking the maximum length of command line arguments... 8192
checking how to convert x86_64-w64-mingw32 file names to x86_64-w64-mingw32 format... func_convert_file_msys_to_w32
checking how to convert x86_64-w64-mingw32 file names to toolchain format... func_convert_file_msys_to_w32
checking for C:/msys/mingw64/x86_64-w64-mingw32/bin/ld.exe option to reload object files... -r
checking for x86_64-w64-mingw32-objdump... no
checking for objdump... objdump
checking how to recognize dependent libraries... file_magic ^x86 archive import|^x86 DLL
checking for x86_64-w64-mingw32-dlltool... no
checking for dlltool... dlltool
checking how to associate runtime and link libraries... func_cygming_dll_for_implib
checking for x86_64-w64-mingw32-ar... no
checking for ar... ar
checking for archiver @FILE support... @
checking for x86_64-w64-mingw32-strip... no
checking for strip... strip
checking for x86_64-w64-mingw32-ranlib... no
checking for ranlib... ranlib
checking command to parse /mingw64/bin/nm -B output from x86_64-w64-mingw32-gcc object... ok
checking for sysroot... no
checking for a working dd... /usr/bin/dd
checking how to truncate binary pipes... /usr/bin/dd bs=4096 count=1
checking for x86_64-w64-mingw32-mt... no
checking for mt... no
checking if : is a manifest tool... no
checking for dlfcn.h... no
checking for objdir... .libs
checking if x86_64-w64-mingw32-gcc supports -fno-rtti -fno-exceptions... no
checking for x86_64-w64-mingw32-gcc option to produce PIC... -DDLL_EXPORT -DPIC
checking if x86_64-w64-mingw32-gcc PIC flag -DDLL_EXPORT -DPIC works... yes
checking if x86_64-w64-mingw32-gcc static flag -static works... yes
checking if x86_64-w64-mingw32-gcc supports -c -o file.o... yes
checking if x86_64-w64-mingw32-gcc supports -c -o file.o... (cached) yes
checking whether the x86_64-w64-mingw32-gcc linker (C:/msys/mingw64/x86_64-w64-mingw32/bin/ld.exe) supports shared libraries... yes
checking whether -lc should be explicitly linked in... yes
checking dynamic linker characteristics... Win32 ld.exe
checking how to hardcode library paths into programs... immediate
checking whether stripping libraries is possible... yes
checking if libtool supports shared libraries... yes
checking whether to build shared libraries... yes
checking whether to build static libraries... yes
checking whether the -Werror option is usable... yes
checking for simple visibility declarations... yes
checking for ANSI C header files... (cached) yes
checking whether sys/types.h defines makedev... no
checking sys/mkdev.h usability... no
checking sys/mkdev.h presence... no
checking for sys/mkdev.h... no
checking sys/sysmacros.h usability... no
checking sys/sysmacros.h presence... no
checking for sys/sysmacros.h... no
checking for sys/wait.h that is POSIX.1 compatible... no
checking for stdint.h... (cached) yes
checking fcntl.h usability... yes
checking fcntl.h presence... yes
checking for fcntl.h... yes
checking for inttypes.h... (cached) yes
checking for unistd.h... (cached) yes
checking utime.h usability... yes
checking utime.h presence... yes
checking for utime.h... yes
checking wchar.h usability... yes
checking wchar.h presence... yes
checking for wchar.h... yes
checking wctype.h usability... yes
checking wctype.h presence... yes
checking for wctype.h... yes
checking getopt.h usability... yes
checking getopt.h presence... yes
checking for getopt.h... yes
checking err.h usability... no
checking err.h presence... no
checking for err.h... no
checking xlocale.h usability... no
checking xlocale.h presence... no
checking for xlocale.h... no
checking sys/mman.h usability... no
checking sys/mman.h presence... no
checking for sys/mman.h... no
checking for sys/stat.h... (cached) yes
checking for sys/types.h... (cached) yes
checking sys/utime.h usability... yes
checking sys/utime.h presence... yes
checking for sys/utime.h... yes
checking sys/time.h usability... yes
checking sys/time.h presence... yes
checking for sys/time.h... yes
checking for sys/sysmacros.h... (cached) no
checking zlib.h usability... yes
checking zlib.h presence... yes
checking for zlib.h... yes
checking bzlib.h usability... yes
checking bzlib.h presence... yes
checking for bzlib.h... yes
checking lzma.h usability... yes
checking lzma.h presence... yes
checking for lzma.h... yes
checking for sig_t... no
checking for off_t... yes
checking for size_t... yes
checking for struct stat.st_rdev... yes
checking for struct tm.tm_gmtoff... no
checking whether struct tm is in sys/time.h or time.h... time.h
checking for struct tm.tm_zone... no
checking whether tzname is declared... yes
checking for tzname... yes
checking for tm_zone in struct tm... no
checking whether tzname is declared... (cached) yes
checking for tzname... (cached) yes
checking for tm_isdst in struct tm... yes
checking whether daylight is declared... yes
checking for daylight... yes
checking for special C compiler options needed for large files... no
checking for _FILE_OFFSET_BITS value needed for large files... 64
checking for _LARGEFILE_SOURCE value needed for large files... no
checking for mbstate_t... yes
checking for struct option in getopt... yes
checking for pid_t... yes
checking for uint8_t... yes
checking for uint16_t... yes
checking for uint32_t... yes
checking for int32_t... yes
checking for uint64_t... yes
checking for int64_t... yes
checking for intptr_t... yes
checking for uintptr_t... yes
checking for stdlib.h... (cached) yes
checking for unistd.h... (cached) yes
checking for sys/param.h... yes
checking for getpagesize... yes
checking for working mmap... no
checking vfork.h usability... no
checking vfork.h presence... no
checking for vfork.h... no
checking for fork... no
checking for vfork... no
checking whether mbrtowc and mbstate_t are properly declared... yes
checking for gcc compiler warnings... yes
checking for strndup... no
checking for mkstemp... yes
checking for mkostemp... no
checking for utimes... no
checking for utime... yes
checking for wcwidth... no
checking for strtof... yes
checking for newlocale... no
checking for uselocale... no
checking for freelocale... no
checking for memmem... no
checking for getopt_long... yes
checking for asprintf... yes
checking for vasprintf... yes
checking for strlcpy... no
checking for strlcat... no
checking for getline... no
checking for ctime_r... no
checking for asctime_r... no
checking for localtime_r... no
checking for gmtime_r... no
checking for pread... no
checking for strcasestr... no
checking for fmtcheck... no
checking for dprintf... no
checking for gzopen in -lz... yes
checking for BZ2_bzCompressInit in -lbz2... yes
checking for lzma_stream_decoder in -llzma... yes
checking for regexec in -lgnurx... yes
checking that generated files are newer than configure... done
configure: creating ./config.status
config.status: creating Makefile
config.status: creating src/Makefile
config.status: creating magic/Makefile
config.status: creating tests/Makefile
config.status: creating doc/Makefile
config.status: creating python/Makefile
config.status: creating config.h
config.status: executing depfiles commands
config.status: executing libtool commands
make  all-recursive
make[1]: Entering directory '/home/pombreda/MINGW-packages/mingw-w64-file/src/build-x86_64'
Making all in src
make[2]: Entering directory '/home/pombreda/MINGW-packages/mingw-w64-file/src/build-x86_64/src'
sed -e "s/X.YY/$(echo 5.38 | tr -d .)/" < ../../file-5.38/src/magic.h.in > magic.h
make  all-am
make[3]: Entering directory '/home/pombreda/MINGW-packages/mingw-w64-file/src/build-x86_64/src'
  CC       seccomp.o
  CC       file.o
  CC       buffer.lo
  CC       magic.lo
  CC       apprentice.lo
  CC       softmagic.lo
../../file-5.38/src/apprentice.c: In function 'apprentice_1':
../../file-5.38/src/apprentice.c:473:11: warning: format '%u' expects argument of type 'unsigned int', but argument 2 has type 'size_t' {aka 'long long unsigned int'} [-Wformat=]
  473 |    printf("Set %" SIZE_T_FORMAT "u:\nBinary patterns:\n",
      |           ^~~~~~~
  474 |        i);
      |        ~   
      |        |
      |        size_t {aka long long unsigned int}
../../file-5.38/src/apprentice.c:473:34: note: format string is defined here
  473 |    printf("Set %" SIZE_T_FORMAT "u:\nBinary patterns:\n",
      |                ~~~~~~~~~~~~~~~~~~^
      |                                  |
      |                                  unsigned int
      |                %" SIZE_T_FORMAT "llu
../../file-5.38/src/apprentice.c: In function 'apprentice_list':
../../file-5.38/src/apprentice.c:1032:11: warning: unknown conversion type character 'I' in format [-Wformat=]
 1032 |    printf("Strength = %3" SIZE_T_FORMAT "u@%u: %s [%s]\n",
      |           ^~~~~~~~~~~~~~~
In file included from ../../file-5.38/src/apprentice.c:32:
../../file-5.38/src/file.h:49:28: note: format string is defined here
   49 |     #define SIZE_T_FORMAT "I64"
      |                            ^
../../file-5.38/src/apprentice.c:1032:11: warning: format '%u' expects argument of type 'unsigned int', but argument 2 has type 'size_t' {aka 'long long unsigned int'} [-Wformat=]
 1032 |    printf("Strength = %3" SIZE_T_FORMAT "u@%u: %s [%s]\n",
      |           ^~~~~~~~~~~~~~~
 1033 |        apprentice_magic_strength(m),
      |        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~
      |        |
      |        size_t {aka long long unsigned int}
../../file-5.38/src/apprentice.c:1032:45: note: format string is defined here
 1032 |    printf("Strength = %3" SIZE_T_FORMAT "u@%u: %s [%s]\n",
      |                                            ~^
      |                                             |
      |                                             unsigned int
      |                                            %llu
../../file-5.38/src/apprentice.c:1032:11: warning: format '%s' expects argument of type 'char *', but argument 3 has type 'uint32_t' {aka 'unsigned int'} [-Wformat=]
 1032 |    printf("Strength = %3" SIZE_T_FORMAT "u@%u: %s [%s]\n",
      |           ^~~~~~~~~~~~~~~
 1033 |        apprentice_magic_strength(m),
 1034 |        ml->magic[magindex].lineno,
      |        ~~~~~~~~~~~~~~~~~~~~~~~~~~
      |                           |
      |                           uint32_t {aka unsigned int}
../../file-5.38/src/apprentice.c:1032:49: note: format string is defined here
 1032 |    printf("Strength = %3" SIZE_T_FORMAT "u@%u: %s [%s]\n",
      |                                                ~^
      |                                                 |
      |                                                 char *
      |                                                %d
../../file-5.38/src/apprentice.c:1032:11: warning: too many arguments for format [-Wformat-extra-args]
 1032 |    printf("Strength = %3" SIZE_T_FORMAT "u@%u: %s [%s]\n",
      |           ^~~~~~~~~~~~~~~
../../file-5.38/src/apprentice.c: In function 'getvalue':
../../file-5.38/src/apprentice.c:2724:22: warning: unknown conversion type character 'l' in format [-Wformat=]
 2724 |     file_magwarn(ms, "Overflow for numeric"
      |                      ^~~~~~~~~~~~~~~~~~~~~~
In file included from ../../file-5.38/src/file.h:67,
                 from ../../file-5.38/src/apprentice.c:32:
C:/msys/mingw64/x86_64-w64-mingw32/include/inttypes.h:37:18: note: format string is defined here
   37 | #define PRIx64 "llx"
      |                  ^
../../file-5.38/src/apprentice.c:2724:22: warning: too many arguments for format [-Wformat-extra-args]
 2724 |     file_magwarn(ms, "Overflow for numeric"
      |                      ^~~~~~~~~~~~~~~~~~~~~~
  CC       ascmagic.lo
  CC       encoding.lo
../../file-5.38/src/softmagic.c: In function 'mprint':
../../file-5.38/src/softmagic.c:650:37: warning: format '%u' expects argument of type 'unsigned int', but argument 4 has type 'long long unsigned int' [-Wformat=]
  650 |    (void)snprintf(buf, sizeof(buf), "%" INT64_T_FORMAT "u",
      |                                     ^~~
../../file-5.38/src/softmagic.c:650:57: note: format string is defined here
  650 |    (void)snprintf(buf, sizeof(buf), "%" INT64_T_FORMAT "u",
      |                                      ~~~~~~~~~~~~~~~~~~~^
      |                                                         |
      |                                                         unsigned int
      |                                      %" INT64_T_FORMAT "llu
../../file-5.38/src/softmagic.c: In function 'moffset':
../../file-5.38/src/softmagic.c:959:10: warning: format '%u' expects argument of type 'unsigned int', but argument 4 has type 'size_t' {aka 'long long unsigned int'} [-Wformat=]
  959 |          "Bad DER offset %d nbytes=%"
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
  960 |          SIZE_T_FORMAT "u", o, nbytes);
      |                                ~~~~~~
      |                                |
      |                                size_t {aka long long unsigned int}
../../file-5.38/src/softmagic.c:960:25: note: format string is defined here
  959 |          "Bad DER offset %d nbytes=%"
      |                                    ~~
  960 |          SIZE_T_FORMAT "u", o, nbytes);
      |          ~~~~~~~~~~~~~~~^
      |                         |
      |                         unsigned int
../../file-5.38/src/softmagic.c: In function 'mdebug':
../../file-5.38/src/softmagic.c:1298:25: warning: format '%u' expects argument of type 'unsigned int', but argument 3 has type 'size_t' {aka 'long long unsigned int'} [-Wformat=]
 1298 |  (void) fprintf(stderr, "mget/%" SIZE_T_FORMAT "u @%d: ", len, offset);
      |                         ^~~~~~~~                          ~~~
      |                                                           |
      |                                                           size_t {aka long long unsigned int}
../../file-5.38/src/softmagic.c:1298:49: note: format string is defined here
 1298 |  (void) fprintf(stderr, "mget/%" SIZE_T_FORMAT "u @%d: ", len, offset);
      |                               ~~~~~~~~~~~~~~~~~~^
      |                                                 |
      |                                                 unsigned int
      |                               %" SIZE_T_FORMAT "llu
../../file-5.38/src/softmagic.c: In function 'msetoffset':
../../file-5.38/src/softmagic.c:1508:19: warning: format '%u' expects argument of type 'unsigned int', but argument 4 has type 'size_t' {aka 'long long unsigned int'} [-Wformat=]
 1508 |   fprintf(stderr, "bb=[%p,%" SIZE_T_FORMAT "u], %d [b=%p,%"
      |                   ^~~~~~~~~~
 1509 |       SIZE_T_FORMAT "u], [o=%#x, c=%d]\n",
 1510 |       bb->fbuf, bb->flen, ms->offset, b->fbuf, b->flen,
      |                 ~~~~~~~~
      |                   |
      |                   size_t {aka long long unsigned int}
../../file-5.38/src/softmagic.c:1508:45: note: format string is defined here
 1508 |   fprintf(stderr, "bb=[%p,%" SIZE_T_FORMAT "u], %d [b=%p,%"
      |                           ~~~~~~~~~~~~~~~~~~^
      |                                             |
      |                                             unsigned int
      |                           %" SIZE_T_FORMAT "llu
../../file-5.38/src/softmagic.c:1508:19: warning: format '%u' expects argument of type 'unsigned int', but argument 7 has type 'size_t' {aka 'const long long unsigned int'} [-Wformat=]
 1508 |   fprintf(stderr, "bb=[%p,%" SIZE_T_FORMAT "u], %d [b=%p,%"
      |                   ^~~~~~~~~~
 1509 |       SIZE_T_FORMAT "u], [o=%#x, c=%d]\n",
 1510 |       bb->fbuf, bb->flen, ms->offset, b->fbuf, b->flen,
      |                                                ~~~~~~~
      |                                                 |
      |                                                 size_t {aka const long long unsigned int}
../../file-5.38/src/softmagic.c:1509:22: note: format string is defined here
 1508 |   fprintf(stderr, "bb=[%p,%" SIZE_T_FORMAT "u], %d [b=%p,%"
      |                                                          ~~
 1509 |       SIZE_T_FORMAT "u], [o=%#x, c=%d]\n",
      |       ~~~~~~~~~~~~~~~^
      |                      |
      |                      unsigned int
../../file-5.38/src/softmagic.c: In function 'mget':
../../file-5.38/src/softmagic.c:1551:19: warning: format '%u' expects argument of type 'unsigned int', but argument 6 has type 'size_t' {aka 'long long unsigned int'} [-Wformat=]
 1551 |   fprintf(stderr, "mget(type=%d, flag=%#x, offset=%u, o=%"
      |                   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
......
 1554 |       m->type, m->flag, offset, o, nbytes,
      |                                 ~
      |                                 |
      |                                 size_t {aka long long unsigned int}
../../file-5.38/src/softmagic.c:1552:22: note: format string is defined here
 1551 |   fprintf(stderr, "mget(type=%d, flag=%#x, offset=%u, o=%"
      |                                                         ~~
 1552 |       SIZE_T_FORMAT "u, " "nbytes=%" SIZE_T_FORMAT
      |       ~~~~~~~~~~~~~~~^
      |                      |
      |                      unsigned int
../../file-5.38/src/softmagic.c:1551:19: warning: format '%u' expects argument of type 'unsigned int', but argument 7 has type 'size_t' {aka 'long long unsigned int'} [-Wformat=]
 1551 |   fprintf(stderr, "mget(type=%d, flag=%#x, offset=%u, o=%"
      |                   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
......
 1554 |       m->type, m->flag, offset, o, nbytes,
      |                                    ~~~~~~
      |                                    |
      |                                    size_t {aka long long unsigned int}
../../file-5.38/src/softmagic.c:1553:8: note: format string is defined here
 1552 |       SIZE_T_FORMAT "u, " "nbytes=%" SIZE_T_FORMAT
      |                                   ~~~~~~~~~~~~~~~~
 1553 |       "u, il=%hu, nc=%hu)\n",
      |       ~^
      |        |
      |        unsigned int
../../file-5.38/src/softmagic.c: In function 'magiccheck':
../../file-5.38/src/softmagic.c:2192:27: warning: format '%u' expects argument of type 'unsigned int', but argument 3 has type 'long long unsigned int' [-Wformat=]
 2192 |    (void) fprintf(stderr, "%" INT64_T_FORMAT
      |                           ^~~
../../file-5.38/src/softmagic.c:2193:9: note: format string is defined here
 2192 |    (void) fprintf(stderr, "%" INT64_T_FORMAT
      |                            ~~~~~~~~~~~~~~~~~
 2193 |        "u == *any* = 1\n", CAST(unsigned long long, v));
      |        ~^
      |         |
      |         unsigned int
../../file-5.38/src/softmagic.c:2200:27: warning: format '%u' expects argument of type 'unsigned int', but argument 3 has type 'long long unsigned int' [-Wformat=]
 2200 |    (void) fprintf(stderr, "%" INT64_T_FORMAT "u != %"
      |                           ^~~
../../file-5.38/src/softmagic.c:2200:47: note: format string is defined here
 2200 |    (void) fprintf(stderr, "%" INT64_T_FORMAT "u != %"
      |                            ~~~~~~~~~~~~~~~~~~~^
      |                                               |
      |                                               unsigned int
      |                            %" INT64_T_FORMAT "llu
../../file-5.38/src/softmagic.c:2200:27: warning: format '%u' expects argument of type 'unsigned int', but argument 4 has type 'long long unsigned int' [-Wformat=]
 2200 |    (void) fprintf(stderr, "%" INT64_T_FORMAT "u != %"
      |                           ^~~
../../file-5.38/src/softmagic.c:2201:24: note: format string is defined here
 2200 |    (void) fprintf(stderr, "%" INT64_T_FORMAT "u != %"
      |                                                    ~~
 2201 |        INT64_T_FORMAT "u = %d\n",
      |        ~~~~~~~~~~~~~~~~^
      |                        |
      |                        unsigned int
../../file-5.38/src/softmagic.c:2209:27: warning: format '%u' expects argument of type 'unsigned int', but argument 3 has type 'long long unsigned int' [-Wformat=]
 2209 |    (void) fprintf(stderr, "%" INT64_T_FORMAT "u == %"
      |                           ^~~
../../file-5.38/src/softmagic.c:2209:47: note: format string is defined here
 2209 |    (void) fprintf(stderr, "%" INT64_T_FORMAT "u == %"
      |                            ~~~~~~~~~~~~~~~~~~~^
      |                                               |
      |                                               unsigned int
      |                            %" INT64_T_FORMAT "llu
../../file-5.38/src/softmagic.c:2209:27: warning: format '%u' expects argument of type 'unsigned int', but argument 4 has type 'long long unsigned int' [-Wformat=]
 2209 |    (void) fprintf(stderr, "%" INT64_T_FORMAT "u == %"
      |                           ^~~
../../file-5.38/src/softmagic.c:2210:24: note: format string is defined here
 2209 |    (void) fprintf(stderr, "%" INT64_T_FORMAT "u == %"
      |                                                    ~~
 2210 |        INT64_T_FORMAT "u = %d\n",
      |        ~~~~~~~~~~~~~~~~^
      |                        |
      |                        unsigned int
../../file-5.38/src/softmagic.c:2219:28: warning: format '%u' expects argument of type 'unsigned int', but argument 3 has type 'long long unsigned int' [-Wformat=]
 2219 |     (void) fprintf(stderr, "%" INT64_T_FORMAT
      |                            ^~~
../../file-5.38/src/softmagic.c:2220:10: note: format string is defined here
 2219 |     (void) fprintf(stderr, "%" INT64_T_FORMAT
      |                             ~~~~~~~~~~~~~~~~~
 2220 |         "u > %" INT64_T_FORMAT "u = %d\n",
      |         ~^
      |          |
      |          unsigned int
../../file-5.38/src/softmagic.c:2219:28: warning: format '%u' expects argument of type 'unsigned int', but argument 4 has type 'long long unsigned int' [-Wformat=]
 2219 |     (void) fprintf(stderr, "%" INT64_T_FORMAT
      |                            ^~~
../../file-5.38/src/softmagic.c:2220:33: note: format string is defined here
 2220 |         "u > %" INT64_T_FORMAT "u = %d\n",
      |              ~~~~~~~~~~~~~~~~~~~^
      |                                 |
      |                                 unsigned int
      |              %" INT64_T_FORMAT "llu
../../file-5.38/src/softmagic.c:2227:28: warning: format '%d' expects argument of type 'int', but argument 3 has type 'long long int' [-Wformat=]
 2227 |     (void) fprintf(stderr, "%" INT64_T_FORMAT
      |                            ^~~
../../file-5.38/src/softmagic.c:2228:10: note: format string is defined here
 2227 |     (void) fprintf(stderr, "%" INT64_T_FORMAT
      |                             ~~~~~~~~~~~~~~~~~
 2228 |         "d > %" INT64_T_FORMAT "d = %d\n",
      |         ~^
      |          |
      |          int
../../file-5.38/src/softmagic.c:2227:28: warning: format '%d' expects argument of type 'int', but argument 4 has type 'long long int' [-Wformat=]
 2227 |     (void) fprintf(stderr, "%" INT64_T_FORMAT
      |                            ^~~
../../file-5.38/src/softmagic.c:2228:33: note: format string is defined here
 2228 |         "d > %" INT64_T_FORMAT "d = %d\n",
      |              ~~~~~~~~~~~~~~~~~~~^
      |                                 |
      |                                 int
      |              %" INT64_T_FORMAT "lld
../../file-5.38/src/softmagic.c:2238:28: warning: format '%u' expects argument of type 'unsigned int', but argument 3 has type 'long long unsigned int' [-Wformat=]
 2238 |     (void) fprintf(stderr, "%" INT64_T_FORMAT
      |                            ^~~
../../file-5.38/src/softmagic.c:2239:10: note: format string is defined here
 2238 |     (void) fprintf(stderr, "%" INT64_T_FORMAT
      |                             ~~~~~~~~~~~~~~~~~
 2239 |         "u < %" INT64_T_FORMAT "u = %d\n",
      |         ~^
      |          |
      |          unsigned int
../../file-5.38/src/softmagic.c:2238:28: warning: format '%u' expects argument of type 'unsigned int', but argument 4 has type 'long long unsigned int' [-Wformat=]
 2238 |     (void) fprintf(stderr, "%" INT64_T_FORMAT
      |                            ^~~
../../file-5.38/src/softmagic.c:2239:33: note: format string is defined here
 2239 |         "u < %" INT64_T_FORMAT "u = %d\n",
      |              ~~~~~~~~~~~~~~~~~~~^
      |                                 |
      |                                 unsigned int
      |              %" INT64_T_FORMAT "llu
../../file-5.38/src/softmagic.c:2246:28: warning: format '%d' expects argument of type 'int', but argument 3 has type 'long long int' [-Wformat=]
 2246 |     (void) fprintf(stderr, "%" INT64_T_FORMAT
      |                            ^~~
../../file-5.38/src/softmagic.c:2247:10: note: format string is defined here
 2246 |     (void) fprintf(stderr, "%" INT64_T_FORMAT
      |                             ~~~~~~~~~~~~~~~~~
 2247 |         "d < %" INT64_T_FORMAT "d = %d\n",
      |         ~^
      |          |
      |          int
../../file-5.38/src/softmagic.c:2246:28: warning: format '%d' expects argument of type 'int', but argument 4 has type 'long long int' [-Wformat=]
 2246 |     (void) fprintf(stderr, "%" INT64_T_FORMAT
      |                            ^~~
../../file-5.38/src/softmagic.c:2247:33: note: format string is defined here
 2247 |         "d < %" INT64_T_FORMAT "d = %d\n",
      |              ~~~~~~~~~~~~~~~~~~~^
      |                                 |
      |                                 int
      |              %" INT64_T_FORMAT "lld
../../file-5.38/src/softmagic.c:2256:27: warning: 'I' flag used with '%x' gnu_printf format [-Wformat=]
 2256 |    (void) fprintf(stderr, "((%" INT64_T_FORMAT "x & %"
      |                           ^~~~~
../../file-5.38/src/softmagic.c:2256:49: note: format string is defined here
 2256 |    (void) fprintf(stderr, "((%" INT64_T_FORMAT "x & %"
      |                                                 ^
../../file-5.38/src/softmagic.c:2256:27: warning: format '%x' expects argument of type 'unsigned int', but argument 3 has type 'long long unsigned int' [-Wformat=]
 2256 |    (void) fprintf(stderr, "((%" INT64_T_FORMAT "x & %"
      |                           ^~~~~
../../file-5.38/src/softmagic.c:2256:49: note: format string is defined here
 2256 |    (void) fprintf(stderr, "((%" INT64_T_FORMAT "x & %"
      |                              ~~~~~~~~~~~~~~~~~~~^
      |                                                 |
      |                                                 unsigned int
      |                              %" INT64_T_FORMAT "llx
../../file-5.38/src/softmagic.c:2256:27: warning: 'I' flag used with '%x' gnu_printf format [-Wformat=]
 2256 |    (void) fprintf(stderr, "((%" INT64_T_FORMAT "x & %"
      |                           ^~~~~
../../file-5.38/src/softmagic.c:2257:24: note: format string is defined here
 2257 |        INT64_T_FORMAT "x) == %" INT64_T_FORMAT
      |                        ^
../../file-5.38/src/softmagic.c:2256:27: warning: format '%x' expects argument of type 'unsigned int', but argument 4 has type 'long long unsigned int' [-Wformat=]
 2256 |    (void) fprintf(stderr, "((%" INT64_T_FORMAT "x & %"
      |                           ^~~~~
../../file-5.38/src/softmagic.c:2257:24: note: format string is defined here
 2256 |    (void) fprintf(stderr, "((%" INT64_T_FORMAT "x & %"
      |                                                     ~~
 2257 |        INT64_T_FORMAT "x) == %" INT64_T_FORMAT
      |        ~~~~~~~~~~~~~~~~^
      |                        |
      |                        unsigned int
../../file-5.38/src/softmagic.c:2256:27: warning: 'I' flag used with '%x' gnu_printf format [-Wformat=]
 2256 |    (void) fprintf(stderr, "((%" INT64_T_FORMAT "x & %"
      |                           ^~~~~
../../file-5.38/src/softmagic.c:2258:9: note: format string is defined here
 2258 |        "x) = %d\n", CAST(unsigned long long, v),
      |         ^
../../file-5.38/src/softmagic.c:2256:27: warning: format '%x' expects argument of type 'unsigned int', but argument 5 has type 'long long unsigned int' [-Wformat=]
 2256 |    (void) fprintf(stderr, "((%" INT64_T_FORMAT "x & %"
      |                           ^~~~~
../../file-5.38/src/softmagic.c:2258:9: note: format string is defined here
 2257 |        INT64_T_FORMAT "x) == %" INT64_T_FORMAT
      |                              ~~~~~~~~~~~~~~~~~
 2258 |        "x) = %d\n", CAST(unsigned long long, v),
      |        ~^
      |         |
      |         unsigned int
../../file-5.38/src/softmagic.c:2267:27: warning: 'I' flag used with '%x' gnu_printf format [-Wformat=]
 2267 |    (void) fprintf(stderr, "((%" INT64_T_FORMAT "x & %"
      |                           ^~~~~
../../file-5.38/src/softmagic.c:2267:49: note: format string is defined here
 2267 |    (void) fprintf(stderr, "((%" INT64_T_FORMAT "x & %"
      |                                                 ^
../../file-5.38/src/softmagic.c:2267:27: warning: format '%x' expects argument of type 'unsigned int', but argument 3 has type 'long long unsigned int' [-Wformat=]
 2267 |    (void) fprintf(stderr, "((%" INT64_T_FORMAT "x & %"
      |                           ^~~~~
../../file-5.38/src/softmagic.c:2267:49: note: format string is defined here
 2267 |    (void) fprintf(stderr, "((%" INT64_T_FORMAT "x & %"
      |                              ~~~~~~~~~~~~~~~~~~~^
      |                                                 |
      |                                                 unsigned int
      |                              %" INT64_T_FORMAT "llx
../../file-5.38/src/softmagic.c:2267:27: warning: 'I' flag used with '%x' gnu_printf format [-Wformat=]
 2267 |    (void) fprintf(stderr, "((%" INT64_T_FORMAT "x & %"
      |                           ^~~~~
../../file-5.38/src/softmagic.c:2268:24: note: format string is defined here
 2268 |        INT64_T_FORMAT "x) != %" INT64_T_FORMAT
      |                        ^
../../file-5.38/src/softmagic.c:2267:27: warning: format '%x' expects argument of type 'unsigned int', but argument 4 has type 'long long unsigned int' [-Wformat=]
 2267 |    (void) fprintf(stderr, "((%" INT64_T_FORMAT "x & %"
      |                           ^~~~~
../../file-5.38/src/softmagic.c:2268:24: note: format string is defined here
 2267 |    (void) fprintf(stderr, "((%" INT64_T_FORMAT "x & %"
      |                                                     ~~
 2268 |        INT64_T_FORMAT "x) != %" INT64_T_FORMAT
      |        ~~~~~~~~~~~~~~~~^
      |                        |
      |                        unsigned int
../../file-5.38/src/softmagic.c:2267:27: warning: 'I' flag used with '%x' gnu_printf format [-Wformat=]
 2267 |    (void) fprintf(stderr, "((%" INT64_T_FORMAT "x & %"
      |                           ^~~~~
../../file-5.38/src/softmagic.c:2269:9: note: format string is defined here
 2269 |        "x) = %d\n", CAST(unsigned long long, v),
      |         ^
../../file-5.38/src/softmagic.c:2267:27: warning: format '%x' expects argument of type 'unsigned int', but argument 5 has type 'long long unsigned int' [-Wformat=]
 2267 |    (void) fprintf(stderr, "((%" INT64_T_FORMAT "x & %"
      |                           ^~~~~
../../file-5.38/src/softmagic.c:2269:9: note: format string is defined here
 2268 |        INT64_T_FORMAT "x) != %" INT64_T_FORMAT
      |                              ~~~~~~~~~~~~~~~~~
 2269 |        "x) = %d\n", CAST(unsigned long long, v),
      |        ~^
      |         |
      |         unsigned int
  CC       compress.lo
  CC       is_csv.lo
  CC       is_json.lo
../../file-5.38/src/compress.c: In function 'sread':
../../file-5.38/src/compress.c:390:20: warning: implicit declaration of function 'ioctl'; did you mean 'iscntrl'? [-Wimplicit-function-declaration]
  390 |  if (canbepipe && (ioctl(fd, FIONREAD, &t) == -1 || t == 0)) {
      |                    ^~~~~
      |                    iscntrl
../../file-5.38/src/compress.c:390:20: warning: nested extern declaration of 'ioctl' [-Wnested-externs]
At top level:
../../file-5.38/src/compress.c:168:3: warning: 'compr' defined but not used [-Wunused-const-variable=]
  168 | } compr[] = {
      |   ^~~~~
  CC       is_tar.lo
  CC       readelf.lo
  CC       print.lo
../../file-5.38/src/readelf.c: In function 'doshn':
../../file-5.38/src/readelf.c:1352:59: warning: unknown conversion type character 't' in format [-Wformat=]
 1352 |   if (file_printf(ms, ", too large section header offset %td",
      |                                                           ^
../../file-5.38/src/readelf.c:1352:23: warning: too many arguments for format [-Wformat-extra-args]
 1352 |   if (file_printf(ms, ", too large section header offset %td",
      |                       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  CC       fsmagic.lo
  CC       funcs.lo
../../file-5.38/src/print.c: In function 'file_mdump':
../../file-5.38/src/print.c:142:27: warning: format '%d' expects argument of type 'int', but argument 3 has type 'long long int' [-Wformat=]
  142 |    (void) fprintf(stderr, "%" INT64_T_FORMAT "d",
      |                           ^~~
../../file-5.38/src/print.c:142:47: note: format string is defined here
  142 |    (void) fprintf(stderr, "%" INT64_T_FORMAT "d",
      |                            ~~~~~~~~~~~~~~~~~~~^
      |                                               |
      |                                               int
      |                            %" INT64_T_FORMAT "lld
  CC       apptype.lo
  CC       der.lo
  CC       cdf.lo
  CC       cdf_time.lo
../../file-5.38/src/der.c: In function 'der_cmp':
../../file-5.38/src/der.c:318:20: warning: format '%u' expects argument of type 'unsigned int', but argument 4 has type 'size_t' {aka 'long long unsigned int'} [-Wformat=]
  318 |    fprintf(stderr, "%s: len %" SIZE_T_FORMAT "u %u\n",
      |                    ^~~~~~~~~~~
  319 |        __func__, slen, tlen);
      |                  ~~~~
      |                  |
      |                  size_t {aka long long unsigned int}
../../file-5.38/src/der.c:318:47: note: format string is defined here
  318 |    fprintf(stderr, "%s: len %" SIZE_T_FORMAT "u %u\n",
      |                             ~~~~~~~~~~~~~~~~~~^
      |                                               |
      |                                               unsigned int
      |                             %" SIZE_T_FORMAT "llu
  CC       readcdf.lo
  CC       strlcpy.lo
../../file-5.38/src/cdf_time.c: In function 'cdf_ctime':
../../file-5.38/src/cdf_time.c:174:26: warning: unknown conversion type character 'I' in format [-Wformat=]
  174 |  (void)snprintf(buf, 26, "*Bad* %#16.16" INT64_T_FORMAT "x\n",
      |                          ^~~~~~~~~~~~~~~
In file included from ../../file-5.38/src/cdf_time.c:27:
../../file-5.38/src/file.h:53:27: note: format string is defined here
   53 |   #define INT64_T_FORMAT "I64"
      |                           ^
../../file-5.38/src/cdf_time.c:174:26: warning: too many arguments for format [-Wformat-extra-args]
  174 |  (void)snprintf(buf, 26, "*Bad* %#16.16" INT64_T_FORMAT "x\n",
      |                          ^~~~~~~~~~~~~~~
../../file-5.38/src/readcdf.c: In function 'cdf_file_property_info':
../../file-5.38/src/readcdf.c:239:20: warning: passing argument 1 of 'cdf_ctime' from incompatible pointer type [-Wincompatible-pointer-types]
  239 |      c = cdf_ctime(&ts.tv_sec, tbuf);
      |                    ^~~~~~~~~~
      |                    |
      |                    long int *
In file included from ../../file-5.38/src/readcdf.c:39:
../../file-5.38/src/cdf.h:337:17: note: expected 'const time_t *' {aka 'const long long int *'} but argument is of type 'long int *'
  337 | char *cdf_ctime(const time_t *, char *);
      |                 ^~~~~~~~~~~~~~
  CC       strlcat.lo
  CC       getline.lo
  CC       ctime_r.lo
  CC       asctime_r.lo
  CC       localtime_r.lo
  CC       gmtime_r.lo
  CC       pread.lo
  CC       strcasestr.lo
  CC       fmtcheck.lo
  CC       dprintf.lo
  CCLD     libmagic.la
C:/msys/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.1.0/../../../../x86_64-w64-mingw32/bin/ld.exe: .libs/compress.o:compress.c:(.text+0xa6): undefined reference to `ioctl'
C:/msys/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.1.0/../../../../x86_64-w64-mingw32/bin/ld.exe: .libs/compress.o:compress.c:(.text+0x101): undefined reference to `__imp_select'
C:/msys/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.1.0/../../../../x86_64-w64-mingw32/bin/ld.exe: .libs/compress.o:compress.c:(.text+0x127): undefined reference to `ioctl'
collect2.exe: error: ld returned 1 exit status
make[3]: *** [Makefile:499: libmagic.la] Error 1
make[3]: Leaving directory '/home/pombreda/MINGW-packages/mingw-w64-file/src/build-x86_64/src'
make[2]: *** [Makefile:380: all] Error 2
make[2]: Leaving directory '/home/pombreda/MINGW-packages/mingw-w64-file/src/build-x86_64/src'
make[1]: *** [Makefile:400: all-recursive] Error 1
make[1]: Leaving directory '/home/pombreda/MINGW-packages/mingw-w64-file/src/build-x86_64'
make: *** [Makefile:332: all] Error 2
==> ERROR:(B A failure occurred in build().(B
    Aborting...(B

pombredanne

2020-05-19 13:51

reporter   ~0003406

Note that 5.37 builds and runs fine

pombredanne

2020-05-19 21:51

reporter   ~0003407

Here is a likely patch solving the problem. I am testing this now:

diff --git a/src/compress.c b/src/compress.c
index 904c215..c82e84b 100644
--- a/src/compress.c
+++ b/src/compress.c
@@ -380,7 +380,7 @@
 sread(int fd, void *buf, size_t n, int canbepipe __attribute__((__unused__)))
 {
    ssize_t rv;
-#ifdef FIONREAD
+#ifdef FIONREAD && !defined(__MINGW32__) && !defined(WIN32)
    int t = 0;
 #endif
    size_t rn = n;
@@ -388,7 +388,7 @@
    if (fd == STDIN_FILENO)
        goto nocheck;
 
-#ifdef FIONREAD
+#ifdef FIONREAD && !defined(__MINGW32__) && !defined(WIN32)
    if (canbepipe && (ioctl(fd, FIONREAD, &t) == -1 || t == 0)) {
 #ifdef FD_ZERO
        ssize_t cnt;

pombredanne

2020-05-20 08:32

reporter   ~0003408

Actually this patch is wrong all the way. Ignore that

pombredanne

2020-05-20 10:59

reporter   ~0003409

This patch is working and all tests pass on msys2. I am crafting a packaging patch too.

diff --git a/src/compress.c b/src/compress.c
index 904c215..a5fdbb0 100644
--- a/src/compress.c
+++ b/src/compress.c
@@ -380,7 +380,7 @@
 sread(int fd, void *buf, size_t n, int canbepipe __attribute__((__unused__)))
 {
     ssize_t rv;
-#ifdef FIONREAD
+#if defined(FIONREAD) && !defined(__MINGW32__) && !defined(WIN32)
     int t = 0;
 #endif
     size_t rn = n;
@@ -388,7 +388,7 @@
     if (fd == STDIN_FILENO)
         goto nocheck;
 
-#ifdef FIONREAD
+#if defined(FIONREAD) && !defined(__MINGW32__) && !defined(WIN32)
     if (canbepipe && (ioctl(fd, FIONREAD, &t) == -1 || t == 0)) {
 #ifdef FD_ZERO
         ssize_t cnt;

pombredanne

2020-05-20 11:54

reporter   ~0003410

FWIW the PR at https://github.com/msys2/MINGW-packages/pull/6495 and all tests now pass on win32 and win64 when built with msys2

pombredanne

2020-05-20 15:57

reporter   ~0003411

And FYI that PR has been accepted and merged there https://github.com/msys2/MINGW-packages/pull/6495

christos

2020-05-31 00:11

manager   ~0003419

Can you try what I commited in compress.c and file.h just now?
/p/file/cvsroot/file/src/compress.c,v <-- compress.c
new revision: 1.127; previous revision: 1.126
/p/file/cvsroot/file/src/file.h,v <-- file.h
new revision: 1.217; previous revision: 1.216

Issue History

Date Modified Username Field Change
2020-05-19 13:48 pombredanne New Issue
2020-05-19 13:48 pombredanne File Added: mingw-w64-file-5.38-1-x86_64-build.log
2020-05-19 13:51 pombredanne Note Added: 0003406
2020-05-19 21:51 pombredanne Note Added: 0003407
2020-05-20 08:32 pombredanne Note Added: 0003408
2020-05-20 10:59 pombredanne Note Added: 0003409
2020-05-20 11:54 pombredanne Note Added: 0003410
2020-05-20 15:57 pombredanne Note Added: 0003411
2020-05-31 00:11 christos Assigned To => christos
2020-05-31 00:11 christos Status new => assigned
2020-05-31 00:11 christos Status assigned => feedback
2020-05-31 00:11 christos Note Added: 0003419