paleolimbot opened a new issue, #1113:
URL: https://github.com/apache/arrow-adbc/issues/1113

   Check page: 
https://www.r-project.org/nosvn/R.check/r-oldrel-windows-x86_64/adbcsqlite-00check.html
   
   I have to refresh my memory on exactly what is going on here. I think oldrel 
is Rtools42, which I thought had SQLite available on the default 
include/library path. Apparently this is not the case, because the auto-vendor 
of sqlite.c seems to have been triggered.
   
   In any case, it seems that whatever gcc is being used is thoroughly confused 
by `PRId64`.
   
   ```
   sqlite.c:1193:16: warning: too many arguments for format 
[-Wformat-extra-args]
     statement_reader.c:185:21: warning: too many arguments for format 
[-Wformat-extra-args]
     statement_reader.c:200:14: warning: too many arguments for format 
[-Wformat-extra-args]
     statement_reader.c:221:21: warning: too many arguments for format 
[-Wformat-extra-args]
     statement_reader.c:323:22: warning: too many arguments for format 
[-Wformat-extra-args]
     statement_reader.c:362:22: warning: too many arguments for format 
[-Wformat-extra-args]
   ```
   
   <details>
   
   ```
   * installing *source* package 'adbcsqlite' ...
   ** package 'adbcsqlite' successfully unpacked and MD5 sums checked
   ** using staged installation
   Checking for --configure-vars INCLUDE_DIR or LIB_DIR
   Testing R CMD SHLIB with   -lsqlite3
   Success!
   Found vendored ADBC
   ** libs
   Warning: this package has a non-empty 'configure.win' file,
   so building only the main architecture
   
   make[1]: Entering directory 
'/d/temp/Rtmp4ykFuV/R.INSTALL2e71862e32fdf/adbcsqlite/src'
   gcc  -I"D:/RCompile/recent/R-4.2.3/include" -DNDEBUG -I../src  
-DADBC_EXPORT=""    -I"d:/rtools42/x86_64-w64-mingw32.static.posix/include"     
-pedantic -Wstrict-prototypes -O2 -Wall  -std=gnu99 -mfpmath=sse -msse2 
-mstackrealign  -c init.c -o init.o
   gcc  -I"D:/RCompile/recent/R-4.2.3/include" -DNDEBUG -I../src  
-DADBC_EXPORT=""    -I"d:/rtools42/x86_64-w64-mingw32.static.posix/include"     
-pedantic -Wstrict-prototypes -O2 -Wall  -std=gnu99 -mfpmath=sse -msse2 
-mstackrealign  -c sqlite.c -o sqlite.o
   sqlite.c: In function 'SqliteStatementExecuteQuery':
   sqlite.c:1193:16: warning: unknown conversion type character 'l' in format 
[-Wformat=]
    1193 |                "[SQLite] Parameter count mismatch: expected %" PRId64
         |                ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   In file included from sqlite.c:21:
   d:\rtools42\x86_64-w64-mingw32.static.posix\include\inttypes.h:33:18: note: 
format string is defined here
      33 | #define PRId64 "lld"
         |                  ^
   sqlite.c:1193:16: warning: unknown conversion type character 'l' in format 
[-Wformat=]
    1193 |                "[SQLite] Parameter count mismatch: expected %" PRId64
         |                ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   In file included from sqlite.c:21:
   d:\rtools42\x86_64-w64-mingw32.static.posix\include\inttypes.h:33:18: note: 
format string is defined here
      33 | #define PRId64 "lld"
         |                  ^
   sqlite.c:1193:16: warning: too many arguments for format 
[-Wformat-extra-args]
    1193 |                "[SQLite] Parameter count mismatch: expected %" PRId64
         |                ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   gcc  -I"D:/RCompile/recent/R-4.2.3/include" -DNDEBUG -I../src  
-DADBC_EXPORT=""    -I"d:/rtools42/x86_64-w64-mingw32.static.posix/include"     
-pedantic -Wstrict-prototypes -O2 -Wall  -std=gnu99 -mfpmath=sse -msse2 
-mstackrealign  -c statement_reader.c -o statement_reader.o
   statement_reader.c: In function 'ArrowTimestampToIsoString':
   statement_reader.c:185:21: warning: unknown conversion type character 'l' in 
format [-Wformat=]
     185 |     SetError(error, "Timestamp %" PRId64 " with unit %d exceeds 
platform time_t bounds",
         |                     ^~~~~~~~~~~~~
   In file included from statement_reader.c:21:
   d:\rtools42\x86_64-w64-mingw32.static.posix\include\inttypes.h:33:18: note: 
format string is defined here
      33 | #define PRId64 "lld"
         |                  ^
   statement_reader.c:185:21: warning: format '%d' expects argument of type 
'int', but argument 3 has type 'int64_t' {aka 'long long int'} [-Wformat=]
     185 |     SetError(error, "Timestamp %" PRId64 " with unit %d exceeds 
platform time_t bounds",
         |                     ^~~~~~~~~~~~~
     186 |              value, unit);
         |              ~~~~~   
         |              |
         |              int64_t {aka long long int}
   statement_reader.c:185:55: note: format string is defined here
     185 |     SetError(error, "Timestamp %" PRId64 " with unit %d exceeds 
platform time_t bounds",
         |                                                      ~^
         |                                                       |
         |                                                       int
         |                                                      %I64d
   statement_reader.c:185:21: warning: too many arguments for format 
[-Wformat-extra-args]
     185 |     SetError(error, "Timestamp %" PRId64 " with unit %d exceeds 
platform time_t bounds",
         |                     ^~~~~~~~~~~~~
   statement_reader.c:200:14: warning: unknown conversion type character 'l' in 
format [-Wformat=]
     200 |              "Could not convert timestamp %" PRId64 " with unit %d 
to broken down time",
         |              ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   In file included from statement_reader.c:21:
   d:\rtools42\x86_64-w64-mingw32.static.posix\include\inttypes.h:33:18: note: 
format string is defined here
      33 | #define PRId64 "lld"
         |                  ^
   statement_reader.c:200:14: warning: format '%d' expects argument of type 
'int', but argument 3 has type 'int64_t' {aka 'long long int'} [-Wformat=]
     200 |              "Could not convert timestamp %" PRId64 " with unit %d 
to broken down time",
         |              ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     201 |              value, unit);
         |              ~~~~~
         |              |
         |              int64_t {aka long long int}
   statement_reader.c:200:66: note: format string is defined here
     200 |              "Could not convert timestamp %" PRId64 " with unit %d 
to broken down time",
         |                                                                 ~^
         |                                                                  |
         |                                                                  int
         |                                                                 %I64d
   statement_reader.c:200:14: warning: too many arguments for format 
[-Wformat-extra-args]
     200 |              "Could not convert timestamp %" PRId64 " with unit %d 
to broken down time",
         |              ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   statement_reader.c:221:21: warning: unknown conversion type character 'l' in 
format [-Wformat=]
     221 |     SetError(error, "Call to strftime for timestamp %" PRId64 " with 
unit %d failed",
         |                     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   In file included from statement_reader.c:21:
   d:\rtools42\x86_64-w64-mingw32.static.posix\include\inttypes.h:33:18: note: 
format string is defined here
      33 | #define PRId64 "lld"
         |                  ^
   statement_reader.c:221:21: warning: format '%d' expects argument of type 
'int', but argument 3 has type 'int64_t' {aka 'long long int'} [-Wformat=]
     221 |     SetError(error, "Call to strftime for timestamp %" PRId64 " with 
unit %d failed",
         |                     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     222 |              value, unit);
         |              ~~~~~   
         |              |
         |              int64_t {aka long long int}
   statement_reader.c:221:76: note: format string is defined here
     221 |     SetError(error, "Call to strftime for timestamp %" PRId64 " with 
unit %d failed",
         |                                                                      
     ~^
         |                                                                      
      |
         |                                                                      
      int
         |                                                                      
     %I64d
   statement_reader.c:221:21: warning: too many arguments for format 
[-Wformat-extra-args]
     221 |     SetError(error, "Call to strftime for timestamp %" PRId64 " with 
unit %d failed",
         |                     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   statement_reader.c: In function 'AdbcSqliteBinderBindNext':
   statement_reader.c:323:22: warning: unknown conversion type character 'l' in 
format [-Wformat=]
     323 |                      "Column %d has unsigned integer value %" PRIu64
         |                      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   In file included from statement_reader.c:21:
   d:\rtools42\x86_64-w64-mingw32.static.posix\include\inttypes.h:36:18: note: 
format string is defined here
      36 | #define PRIu64 "llu"
         |                  ^
   statement_reader.c:323:22: warning: too many arguments for format 
[-Wformat-extra-args]
     323 |                      "Column %d has unsigned integer value %" PRIu64
         |                      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   statement_reader.c:362:22: warning: unknown conversion type character 'l' in 
format [-Wformat=]
     362 |                      "Column %d has value %" PRId64
         |                      ^~~~~~~~~~~~~~~~~~~~~~~
   In file included from statement_reader.c:21:
   d:\rtools42\x86_64-w64-mingw32.static.posix\include\inttypes.h:33:18: note: 
format string is defined here
      33 | #define PRId64 "lld"
         |                  ^
   statement_reader.c:362:22: warning: too many arguments for format 
[-Wformat-extra-args]
     362 |                      "Column %d has value %" PRId64
         |                      ^~~~~~~~~~~~~~~~~~~~~~~
   gcc  -I"D:/RCompile/recent/R-4.2.3/include" -DNDEBUG -I../src  
-DADBC_EXPORT=""    -I"d:/rtools42/x86_64-w64-mingw32.static.posix/include"     
-pedantic -Wstrict-prototypes -O2 -Wall  -std=gnu99 -mfpmath=sse -msse2 
-mstackrealign  -c common/utils.c -o common/utils.o
   gcc  -I"D:/RCompile/recent/R-4.2.3/include" -DNDEBUG -I../src  
-DADBC_EXPORT=""    -I"d:/rtools42/x86_64-w64-mingw32.static.posix/include"     
-pedantic -Wstrict-prototypes -O2 -Wall  -std=gnu99 -mfpmath=sse -msse2 
-mstackrealign  -c nanoarrow/nanoarrow.c -o nanoarrow/nanoarrow.o
   gcc -shared -s -static-libgcc -o adbcsqlite.dll tmp.def init.o sqlite.o 
statement_reader.o common/utils.o nanoarrow/nanoarrow.o -lsqlite3 
-Ld:/rtools42/x86_64-w64-mingw32.static.posix/lib/x64 
-Ld:/rtools42/x86_64-w64-mingw32.static.posix/lib 
-LD:/RCompile/recent/R-4.2.3/bin/x64 -lR
   make[1]: Leaving directory 
'/d/temp/Rtmp4ykFuV/R.INSTALL2e71862e32fdf/adbcsqlite/src'
   make[1]: Entering directory 
'/d/temp/Rtmp4ykFuV/R.INSTALL2e71862e32fdf/adbcsqlite/src'
   make[1]: Leaving directory 
'/d/temp/Rtmp4ykFuV/R.INSTALL2e71862e32fdf/adbcsqlite/src'
   installing to 
d:/Rcompile/CRANpkg/lib/4.2/00LOCK-adbcsqlite/00new/adbcsqlite/libs/x64
   ** R
   ** byte-compile and prepare package for lazy loading
   ** help
   *** installing help indices
   ** building package indices
   ** testing if installed package can be loaded from temporary location
   ** testing if installed package can be loaded from final location
   ** testing if installed package keeps a record of temporary installation path
   * MD5 sums
   packaged installation of 'adbcsqlite' as adbcsqlite_0.6.0.zip
   * DONE (adbcsqlite)
   ```
   
   </details>


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to