citusdata / test-automation

Tools for making our tests easier to run
14 stars 5 forks source link

tpch_2_13_0/update_release.sh prompts warnings while building #184

Open onurctirtir opened 4 years ago

onurctirtir commented 4 years ago

We get mostly int/long int gcc warnings (and more) from update_release.sh when using the script in jdbc tests (https://github.com/citusdata/test-automation/tree/jdbc).

We should investigate the problem a bit closer, maybe we can update the tpch version or fix the way that script triggers the build

./update_release.sh 2 13 0
gcc -g -DDBNAME=\"dss\" -DLINUX -DORACLE -DTPCH -DRNG_TEST -D_FILE_OFFSET_BITS=64    -c -o build.o build.c 
In file included from build.c:45:0: 
build.c: In function ‘mk_time’: 
dss.h:487:14: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘long int’ [-Wformat=] 
 sprintf(tgt, "19%02d-%02d-%02d", yr, mn, dy) 
              ^ 
build.c:424:2: note: in expansion of macro ‘PR_DATE’ 
  PR_DATE(t->alpha, y, m, 
  ^ 
dss.h:487:14: warning: format ‘%d’ expects argument of type ‘int’, but argument 4 has type ‘long int’ [-Wformat=] 
 sprintf(tgt, "19%02d-%02d-%02d", yr, mn, dy) 
              ^ 
build.c:424:2: note: in expansion of macro ‘PR_DATE’ 
  PR_DATE(t->alpha, y, m, 
  ^ 
dss.h:487:14: warning: format ‘%d’ expects argument of type ‘int’, but argument 5 has type ‘long int’ [-Wformat=] 
 sprintf(tgt, "19%02d-%02d-%02d", yr, mn, dy) 
              ^ 
build.c:424:2: note: in expansion of macro ‘PR_DATE’ 
  PR_DATE(t->alpha, y, m, 
  ^ 
gcc -g -DDBNAME=\"dss\" -DLINUX -DORACLE -DTPCH -DRNG_TEST -D_FILE_OFFSET_BITS=64    -c -o driver.o driver. 
driver.c: In function ‘gen_tbl’: 
driver.c:368:11: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 3 has type ‘long long int’ [-Wformat=] 
    printf("\nSeeds for %s at rowcount %ld\n", tdefs[tnum].comment, i); 
           ^ 
driver.c: In function ‘partial’: 
driver.c:428:20: warning: format ‘%d’ expects argument of type ‘int’, but argument 4 has type ‘long int’ [-Wformat=] 
   fprintf (stderr, "\tStarting to load stage %d of %d for %s...", 
                    ^ 
driver.c: In function ‘main’: 
driver.c:754:5: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘long int’ [-Wformat=] 
     "Generating update pair #%d for %s", 
     ^ 
gcc -g -DDBNAME=\"dss\" -DLINUX -DORACLE -DTPCH -DRNG_TEST -D_FILE_OFFSET_BITS=64    -c -o bm_utils.o bm_utils.c 
gcc -g -DDBNAME=\"dss\" -DLINUX -DORACLE -DTPCH -DRNG_TEST -D_FILE_OFFSET_BITS=64    -c -o rnd.o rnd.c 
rnd.c: In function ‘row_stop’: 
rnd.c:131:21: warning: format ‘%d’ expects argument of type ‘int’, but argument 4 has type ‘long long int’ [-Wformat=] 
     fprintf(stderr, "\nSEED CHANGE: seed[%d].usage = %d\n", 
                     ^ 
rnd.c: In function ‘dump_seeds’: 
rnd.c:154:11: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 3 has type ‘long long int’ [-Wformat=] 
    printf("%d(%ld):\t%ld\n", i, Seed[i].nCalls, Seed[i].value); 
           ^ 
rnd.c:154:11: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 4 has type ‘long long int’ [-Wformat=] 
gcc -g -DDBNAME=\"dss\" -DLINUX -DORACLE -DTPCH -DRNG_TEST -D_FILE_OFFSET_BITS=64    -c -o print.o print.c 
print.c: In function ‘dbg_print’: 
print.c:138:19: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 3 has type ‘int’ [-Wformat=] 
   fprintf(target, "%ld.%02ld", dollars, cents); 
                   ^ 
print.c:138:19: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 4 has type ‘int’ [-Wformat=] 
gcc -g -DDBNAME=\"dss\" -DLINUX -DORACLE -DTPCH -DRNG_TEST -D_FILE_OFFSET_BITS=64    -c -o load_stub.o load_stub.c 
gcc -g -DDBNAME=\"dss\" -DLINUX -DORACLE -DTPCH -DRNG_TEST -D_FILE_OFFSET_BITS=64    -c -o bcd2.o bcd2.c 
gcc -g -DDBNAME=\"dss\" -DLINUX -DORACLE -DTPCH -DRNG_TEST -D_FILE_OFFSET_BITS=64    -c -o speed_seed.o speed_seed.c 
gcc -g -DDBNAME=\"dss\" -DLINUX -DORACLE -DTPCH -DRNG_TEST -D_FILE_OFFSET_BITS=64    -c -o text.o text.c 
gcc -g -DDBNAME=\"dss\" -DLINUX -DORACLE -DTPCH -DRNG_TEST -D_FILE_OFFSET_BITS=64    -c -o permute.o perute.c 
gcc -g -DDBNAME=\"dss\" -DLINUX -DORACLE -DTPCH -DRNG_TEST -D_FILE_OFFSET_BITS=64    -c -o rng64.o rng64.c 
gcc -g -DDBNAME=\"dss\" -DLINUX -DORACLE -DTPCH -DRNG_TEST -D_FILE_OFFSET_BITS=64  -O -o dbgen build.o driver.o bm_utils.o rnd.o print.o load_stub.o bcd2.o speed_seed.o text.o permute.o rng64.o -lm 
gcc -g -DDBNAME=\"dss\" -DLINUX -DORACLE -DTPCH -DRNG_TEST -D_FILE_OFFSET_BITS=64    -c -o qgen.o qgen.c 
In file included from qgen.c:41:0: 
qgen.c: In function ‘qsub’: 
tpcd.h:76:19: warning: zero-length gnu_printf format string [-Wformat-zero-length] 
 #define SET_DBASE "" 
                   ^ 
qgen.c:197:38: note: in expansion of macro ‘SET_DBASE’ 
                         fprintf(ofp, SET_DBASE, db_name); 
                                      ^ 
qgen.c: In function ‘main’: 
qgen.c:456:32: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘long int’ [-Wformat=] 
                 sprintf(qname, "%d", SEQUENCE(snum, atoi(av[i]))); 
                                ^ 
qgen.c:463:32: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘long int’ [-Wformat=] 
                 sprintf(qname, "%d", SEQUENCE(snum, i)); 
                                ^