Source: graphviz
Version: 2.42.2-8
Severity: serious
Justification: FTBFS
Tags: trixie sid ftbfs
User: lu...@debian.org
Usertags: ftbfs-20240313 ftbfs-trixie ftbfs-impfuncdef

Hi,

During a rebuild of all packages in sid, your package failed to build
on amd64.

This is most likely caused by a change in dpkg 1.22.6, that enabled
-Werror=implicit-function-declaration. For more information, see
https://wiki.debian.org/qa.debian.org/FTBFS#A2024-03-13_-Werror.3Dimplicit-function-declaration

Relevant part (hopefully):
> gcc -DHAVE_CONFIG_H -I. -I../..  -I../.. -I../../lib/cgraph -I../../lib/graph 
> -I../../lib/cdt -I../../lib/pathplan -I../../lib/pack -I../../lib/neatogen 
> -I../../lib/sparse -I../../lib/gvc -I../../lib/common -I../../lib/ingraphs    
> -g -O2 -Werror=implicit-function-declaration 
> -ffile-prefix-map=/<<PKGBUILDDIR>>=. -fstack-protector-strong 
> -fstack-clash-protection -Wformat -Werror=format-security -fcf-protection 
> -Wall -Wextra -Wmissing-include-dirs -Wswitch-default -Wfloat-equal -Wundef 
> -Wshadow -Wpointer-arith -Wbad-function-cast -Wcast-qual -Wconversion 
> -Wstrict-prototypes -Wmissing-prototypes -Wnested-externs 
> -Wno-unknown-pragmas -Wstrict-prototypes -Wpointer-arith -Wall -ffast-math -c 
> -o gvgen.o gvgen.c
> mmio.c: In function ‘mm_read_unsymmetric_sparse’:
> mmio.c:74:27: warning: conversion to ‘long unsigned int’ from ‘int’ may 
> change the sign of the result [-Wsign-conversion]
>    74 |     I = (int *) malloc(nz * sizeof(int));
>       |                           ^
> mmio.c:75:27: warning: conversion to ‘long unsigned int’ from ‘int’ may 
> change the sign of the result [-Wsign-conversion]
>    75 |     J = (int *) malloc(nz * sizeof(int));
>       |                           ^
> mmio.c:76:32: warning: conversion to ‘long unsigned int’ from ‘int’ may 
> change the sign of the result [-Wsign-conversion]
>    76 |     val = (double *) malloc(nz * sizeof(double));
>       |                                ^
> mmio.c: In function ‘mm_read_banner’:
> mmio.c:130:36: warning: conversion from ‘int’ to ‘char’ may change value 
> [-Wconversion]
>   130 |     for (p = mtx; *p != '\0'; *p = tolower(*p), p++);   /* convert to 
> lower case */
>       |                                    ^~~~~~~
> mmio.c:131:36: warning: conversion from ‘int’ to ‘char’ may change value 
> [-Wconversion]
>   131 |     for (p = crd; *p != '\0'; *p = tolower(*p), p++);
>       |                                    ^~~~~~~
> mmio.c:132:42: warning: conversion from ‘int’ to ‘char’ may change value 
> [-Wconversion]
>   132 |     for (p = data_type; *p != '\0'; *p = tolower(*p), p++);
>       |                                          ^~~~~~~
> mmio.c:133:47: warning: conversion from ‘int’ to ‘char’ may change value 
> [-Wconversion]
>   133 |     for (p = storage_scheme; *p != '\0'; *p = tolower(*p), p++);
>       |                                               ^~~~~~~
> mmio.c: In function ‘mm_read_mtx_crd_data’:
> mmio.c:271:40: warning: unused parameter ‘M’ [-Wunused-parameter]
>   271 | int mm_read_mtx_crd_data(FILE * f, int M, int N, int nz, int I[], int 
> J[],
>       |                                    ~~~~^
> mmio.c:271:47: warning: unused parameter ‘N’ [-Wunused-parameter]
>   271 | int mm_read_mtx_crd_data(FILE * f, int M, int N, int nz, int I[], int 
> J[],
>       |                                           ~~~~^
> mmio.c: At top level:
> mmio.c:336:5: warning: no previous prototype for ‘mm_read_mtx_crd’ 
> [-Wmissing-prototypes]
>   336 | int mm_read_mtx_crd(char *fname, int *M, int *N, int *nz, int **I, 
> int **J,
>       |     ^~~~~~~~~~~~~~~
> mmio.c: In function ‘mm_read_mtx_crd’:
> mmio.c:359:29: warning: conversion to ‘long unsigned int’ from ‘int’ may 
> change the sign of the result [-Wsign-conversion]
>   359 |     *I = (int *) malloc(*nz * sizeof(int));
>       |                             ^
> mmio.c:360:29: warning: conversion to ‘long unsigned int’ from ‘int’ may 
> change the sign of the result [-Wsign-conversion]
>   360 |     *J = (int *) malloc(*nz * sizeof(int));
>       |                             ^
> mmio.c:364:42: warning: conversion to ‘long unsigned int’ from ‘int’ may 
> change the sign of the result [-Wsign-conversion]
>   364 |         *val = (double *) malloc(*nz * 2 * sizeof(double));
>       |                                          ^
> mmio.c:370:38: warning: conversion to ‘long unsigned int’ from ‘int’ may 
> change the sign of the result [-Wsign-conversion]
>   370 |         *val = (double *) malloc(*nz * sizeof(double));
>       |                                      ^
> gvgen.c: In function ‘readPos’:
> gvgen.c:121:9: warning: conversion from ‘long int’ to ‘int’ may change value 
> [-Wconversion]
>   121 |     d = strtol(s, e, 10);
>       |         ^~~~~~
> mmio.c: In function ‘mm_typecode_to_str’:
> mmio.c:463:9: warning: variable ‘error’ set but not used 
> [-Wunused-but-set-variable]
>   463 |     int error = 0;
>       |         ^~~~~
> gvgen.c: In function ‘init’:
> gvgen.c:299:25: warning: conversion from ‘int’ to ‘char’ may change value 
> [-Wconversion]
>   299 |                 errexit(c);
>       |                         ^
> gvgen.c:304:25: warning: conversion from ‘int’ to ‘char’ may change value 
> [-Wconversion]
>   304 |                 errexit(c);
>       |                         ^
> gvgen.c:309:25: warning: conversion from ‘int’ to ‘char’ may change value 
> [-Wconversion]
>   309 |                 errexit(c);
>       |                         ^
> gvgen.c:320:25: warning: conversion from ‘int’ to ‘char’ may change value 
> [-Wconversion]
>   320 |                 errexit(c);
>       |                         ^
> gvgen.c:325:25: warning: conversion from ‘int’ to ‘char’ may change value 
> [-Wconversion]
>   325 |                 errexit(c);
>       |                         ^
> gvgen.c:330:25: warning: conversion from ‘int’ to ‘char’ may change value 
> [-Wconversion]
>   330 |                 errexit(c);
>       |                         ^
> gvgen.c:335:25: warning: conversion from ‘int’ to ‘char’ may change value 
> [-Wconversion]
>   335 |                 errexit(c);
>       |                         ^
> gvgen.c:340:25: warning: conversion from ‘int’ to ‘char’ may change value 
> [-Wconversion]
>   340 |                 errexit(c);
>       |                         ^
> gvgen.c:345:25: warning: conversion from ‘int’ to ‘char’ may change value 
> [-Wconversion]
>   345 |                 errexit(c);
>       |                         ^
> gvgen.c:350:25: warning: conversion from ‘int’ to ‘char’ may change value 
> [-Wconversion]
>   350 |                 errexit(c);
>       |                         ^
> gvgen.c:355:25: warning: conversion from ‘int’ to ‘char’ may change value 
> [-Wconversion]
>   355 |                 errexit(c);
>       |                         ^
> gvgen.c:369:25: warning: conversion from ‘int’ to ‘char’ may change value 
> [-Wconversion]
>   369 |                 errexit(c);
>       |                         ^
> gvgen.c:374:25: warning: conversion from ‘int’ to ‘char’ may change value 
> [-Wconversion]
>   374 |                 errexit(c);
>       |                         ^
> gvgen.c:377:25: warning: conversion from ‘int’ to ‘char’ may change value 
> [-Wconversion]
>   377 |                 errexit(c);
>       |                         ^
> gvgen.c:383:25: warning: conversion from ‘int’ to ‘char’ may change value 
> [-Wconversion]
>   383 |                 errexit(c);
>       |                         ^
> gvgen.c:388:25: warning: conversion from ‘int’ to ‘char’ may change value 
> [-Wconversion]
>   388 |                 errexit(c);
>       |                         ^
> gvgen.c:393:25: warning: conversion from ‘int’ to ‘char’ may change value 
> [-Wconversion]
>   393 |                 errexit(c);
>       |                         ^
> gvgen.c:397:25: warning: conversion from ‘int’ to ‘char’ may change value 
> [-Wconversion]
>   397 |                 errexit(c);
>       |                         ^
> gvgen.c:405:25: warning: conversion from ‘int’ to ‘char’ may change value 
> [-Wconversion]
>   405 |                 errexit(c);
>       |                         ^
> gvgen.c:295:9: warning: switch missing default case [-Wswitch-default]
>   295 |         switch (c) {
>       |         ^~~~~~
> gvgen.c: In function ‘main’:
> gvgen.c:513:13: error: implicit declaration of function ‘makeTetrix’; did you 
> mean ‘makeTree’? [-Werror=implicit-function-declaration]
>   513 |             makeTetrix(opts.graphSize1, ef);
>       |             ^~~~~~~~~~
>       |             makeTree
> gvgen.c:513:13: warning: nested extern declaration of ‘makeTetrix’ 
> [-Wnested-externs]
> gvgen.c: In function ‘init’:
> gvgen.c:315:29: warning: this statement may fall through 
> [-Wimplicit-fallthrough=]
>   315 |             opts->isPartial = 1;
>       |             ~~~~~~~~~~~~~~~~^~~
> gvgen.c:316:9: note: here
>   316 |         case 'g':
>       |         ^~~~
> gcc -DHAVE_CONFIG_H -I. -I../..  -I../.. -I../../lib/cgraph -I../../lib/graph 
> -I../../lib/cdt -I../../lib/pathplan -I../../lib/pack -I../../lib/neatogen 
> -I../../lib/sparse -I../../lib/gvc -I../../lib/common -I../../lib/ingraphs    
> -g -O2 -Werror=implicit-function-declaration 
> -ffile-prefix-map=/<<PKGBUILDDIR>>=. -fstack-protector-strong 
> -fstack-clash-protection -Wformat -Werror=format-security -fcf-protection 
> -Wall -Wextra -Wmissing-include-dirs -Wswitch-default -Wfloat-equal -Wundef 
> -Wshadow -Wpointer-arith -Wbad-function-cast -Wcast-qual -Wconversion 
> -Wstrict-prototypes -Wmissing-prototypes -Wnested-externs 
> -Wno-unknown-pragmas -Wstrict-prototypes -Wpointer-arith -Wall -ffast-math -c 
> -o graph_generator.o graph_generator.c
> graph_generator.c: In function ‘makeTree’:
> graph_generator.c:229:22: warning: conversion to ‘unsigned int’ from ‘int’ 
> may change the sign of the result [-Wsign-conversion]
>   229 |     unsigned int n = (ipow(nary,depth)-1)/(nary-1); /* no. of 
> non-leaf nodes */
>       |                      ^
> graph_generator.c:233:23: warning: comparison of integer expressions of 
> different signedness: ‘unsigned int’ and ‘int’ [-Wsign-compare]
>   233 |         for (j = 0; j < nary; j++) {
>       |                       ^
> graph_generator.c:234:17: warning: conversion to ‘int’ from ‘unsigned int’ 
> may change the sign of the result [-Wsign-conversion]
>   234 |             ef (i, idx++);
>       |                 ^
> graph_generator.c:234:23: warning: conversion to ‘int’ from ‘unsigned int’ 
> may change the sign of the result [-Wsign-conversion]
>   234 |             ef (i, idx++);
>       |                    ~~~^~
> graph_generator.c: In function ‘makeBinaryTree’:
> graph_generator.c:242:22: warning: conversion to ‘unsigned int’ from ‘int’ 
> may change the sign of the result [-Wsign-conversion]
>   242 |     unsigned int n = (1 << depth) - 1;
>       |                      ^
> graph_generator.c:245:13: warning: conversion to ‘int’ from ‘unsigned int’ 
> may change the sign of the result [-Wsign-conversion]
>   245 |         ef( i, 2 * i);
>       |             ^
> graph_generator.c:245:18: warning: conversion to ‘int’ from ‘unsigned int’ 
> may change the sign of the result [-Wsign-conversion]
>   245 |         ef( i, 2 * i);
>       |                ~~^~~
> graph_generator.c:246:13: warning: conversion to ‘int’ from ‘unsigned int’ 
> may change the sign of the result [-Wsign-conversion]
>   246 |         ef( i, 2 * i + 1);
>       |             ^
> graph_generator.c:246:22: warning: conversion to ‘int’ from ‘unsigned int’ 
> may change the sign of the result [-Wsign-conversion]
>   246 |         ef( i, 2 * i + 1);
>       |                ~~~~~~^~~
> graph_generator.c: In function ‘makeSierpinski’:
> graph_generator.c:309:21: warning: conversion to ‘size_t’ {aka ‘long unsigned 
> int’} from ‘int’ may change the sign of the result [-Wsign-conversion]
>   309 |     graph = N_NEW(n + 1, vtx_data);
>       |                     ^
> graph_generator.c:294:38: note: in definition of macro ‘N_NEW’
>   294 | #define N_NEW(n,t)       (t*)calloc((n),sizeof(t))
>       |                                      ^
> graph_generator.c:310:21: warning: conversion to ‘size_t’ {aka ‘long unsigned 
> int’} from ‘int’ may change the sign of the result [-Wsign-conversion]
>   310 |     edges = N_NEW(4 * n, int);
>       |                     ^
> graph_generator.c:294:38: note: in definition of macro ‘N_NEW’
>   294 | #define N_NEW(n,t)       (t*)calloc((n),sizeof(t))
>       |                                      ^
> graph_generator.c:301:9: warning: variable ‘nedges’ set but not used 
> [-Wunused-but-set-variable]
>   301 |     int nedges;
>       |         ^~~~~~
> graph_generator.c: At top level:
> graph_generator.c:383:6: warning: no previous prototype for ‘makeTetrix’ 
> [-Wmissing-prototypes]
>   383 | void makeTetrix(int depth, edgefn ef)
>       |      ^~~~~~~~~~
> graph_generator.c: In function ‘makeTetrix’:
> graph_generator.c:396:21: warning: conversion to ‘size_t’ {aka ‘long unsigned 
> int’} from ‘int’ may change the sign of the result [-Wsign-conversion]
>   396 |     graph = N_NEW(n + 1, vtx_data);
>       |                     ^
> graph_generator.c:294:38: note: in definition of macro ‘N_NEW’
>   294 | #define N_NEW(n,t)       (t*)calloc((n),sizeof(t))
>       |                                      ^
> graph_generator.c:397:21: warning: conversion to ‘size_t’ {aka ‘long unsigned 
> int’} from ‘int’ may change the sign of the result [-Wsign-conversion]
>   397 |     edges = N_NEW(6 * n, int);
>       |                     ^
> graph_generator.c:294:38: note: in definition of macro ‘N_NEW’
>   294 | #define N_NEW(n,t)       (t*)calloc((n),sizeof(t))
>       |                                      ^
> graph_generator.c:388:9: warning: variable ‘nedges’ set but not used 
> [-Wunused-but-set-variable]
>   388 |     int nedges;
>       |         ^~~~~~
> graph_generator.c: In function ‘makeRandom’:
> graph_generator.c:484:11: warning: conversion from ‘time_t’ {aka ‘long int’} 
> to ‘unsigned int’ may change value [-Wconversion]
>   484 |     srand(time(0));
>       |           ^~~~~~~
> graph_generator.c: In function ‘mkTree’:
> graph_generator.c:294:37: warning: conversion to ‘size_t’ {aka ‘long unsigned 
> int’} from ‘int’ may change the sign of the result [-Wsign-conversion]
>   294 | #define N_NEW(n,t)       (t*)calloc((n),sizeof(t))
>       |                                     ^~~
> graph_generator.c:563:13: note: in expansion of macro ‘N_NEW’
>   563 |     tp->p = N_NEW(sz,int);
>       |             ^~~~~
> graph_generator.c: In function ‘mkStack’:
> graph_generator.c:294:37: warning: conversion to ‘size_t’ {aka ‘long unsigned 
> int’} from ‘int’ may change the sign of the result [-Wsign-conversion]
>   294 | #define N_NEW(n,t)       (t*)calloc((n),sizeof(t))
>       |                                     ^~~
> graph_generator.c:646:13: note: in expansion of macro ‘N_NEW’
>   646 |     sp->v = N_NEW(sz,pair);
>       |             ^~~~~
> graph_generator.c: In function ‘genCnt’:
> graph_generator.c:683:22: warning: conversion to ‘size_t’ {aka ‘long unsigned 
> int’} from ‘int’ may change the sign of the result [-Wsign-conversion]
>   683 |     int* T = N_NEW(NN+1,int);
>       |                      ^
> graph_generator.c:294:38: note: in definition of macro ‘N_NEW’
>   294 | #define N_NEW(n,t)       (t*)calloc((n),sizeof(t))
>       |                                      ^
> graph_generator.c: In function ‘genTree’:
> graph_generator.c:715:33: warning: declaration of ‘stack’ shadows a global 
> declaration [-Wshadow]
>   715 | genTree (int NN, int* T, stack* stack, tree_t* TREE)
>       |                          ~~~~~~~^~~~~
> graph_generator.c:639:3: note: shadowed declaration is here
>   639 | } stack;
>       |   ^~~~~
> graph_generator.c:726:17: warning: conversion from ‘double’ to ‘int’ may 
> change value [-Wfloat-conversion]
>   726 |             Z = v*drand();
>       |                 ^
> graph_generator.c: In function ‘makeTreeGen’:
> graph_generator.c:787:11: warning: conversion from ‘time_t’ {aka ‘long int’} 
> to ‘unsigned int’ may change value [-Wconversion]
>   787 |     srand(time(0));
>       |           ^~~~~~~
> gcc -DHAVE_CONFIG_H -I. -I../..  -I../.. -I../../lib/cgraph -I../../lib/graph 
> -I../../lib/cdt -I../../lib/pathplan -I../../lib/pack -I../../lib/neatogen 
> -I../../lib/sparse -I../../lib/gvc -I../../lib/common -I../../lib/ingraphs    
> -g -O2 -Werror=implicit-function-declaration 
> -ffile-prefix-map=/<<PKGBUILDDIR>>=. -fstack-protector-strong 
> -fstack-clash-protection -Wformat -Werror=format-security -fcf-protection 
> -Wall -Wextra -Wmissing-include-dirs -Wswitch-default -Wfloat-equal -Wundef 
> -Wshadow -Wpointer-arith -Wbad-function-cast -Wcast-qual -Wconversion 
> -Wstrict-prototypes -Wmissing-prototypes -Wnested-externs 
> -Wno-unknown-pragmas -Wstrict-prototypes -Wpointer-arith -Wall -ffast-math -c 
> -o gml2gv.o gml2gv.c
> bison -y -dv ../../cmd/tools/gmlparse.y
> flex -i ../../cmd/tools/gmlscan.l
> gcc -DHAVE_CONFIG_H -I. -I../..  -I../.. -I../../lib/cgraph -I../../lib/graph 
> -I../../lib/cdt -I../../lib/pathplan -I../../lib/pack -I../../lib/neatogen 
> -I../../lib/sparse -I../../lib/gvc -I../../lib/common -I../../lib/ingraphs    
> -g -O2 -Werror=implicit-function-declaration 
> -ffile-prefix-map=/<<PKGBUILDDIR>>=. -fstack-protector-strong 
> -fstack-clash-protection -Wformat -Werror=format-security -fcf-protection 
> -Wall -Wextra -Wmissing-include-dirs -Wswitch-default -Wfloat-equal -Wundef 
> -Wshadow -Wpointer-arith -Wbad-function-cast -Wcast-qual -Wconversion 
> -Wstrict-prototypes -Wmissing-prototypes -Wnested-externs 
> -Wno-unknown-pragmas -Wstrict-prototypes -Wpointer-arith -Wall -ffast-math -c 
> -o gv2gml.o gv2gml.c
> /usr/bin/sed "s/yy/gml/g" < lex.yy.c > gmlscan.c
> gml2gv.c: In function ‘initargs’:
> gml2gv.c:109:9: warning: switch missing default case [-Wswitch-default]
>   109 |         switch (c) {
>       |         ^~~~~~
> gcc -DHAVE_CONFIG_H -I. -I../..  -I../.. -I../../lib/cgraph -I../../lib/graph 
> -I../../lib/cdt -I../../lib/pathplan -I../../lib/pack -I../../lib/neatogen 
> -I../../lib/sparse -I../../lib/gvc -I../../lib/common -I../../lib/ingraphs    
> -g -O2 -Werror=implicit-function-declaration 
> -ffile-prefix-map=/<<PKGBUILDDIR>>=. -fstack-protector-strong 
> -fstack-clash-protection -Wformat -Werror=format-security -fcf-protection 
> -Wall -Wextra -Wmissing-include-dirs -Wswitch-default -Wfloat-equal -Wundef 
> -Wshadow -Wpointer-arith -Wbad-function-cast -Wcast-qual -Wconversion 
> -Wstrict-prototypes -Wmissing-prototypes -Wnested-externs 
> -Wno-unknown-pragmas -Wstrict-prototypes -Wpointer-arith -Wall -ffast-math -c 
> -o graphml2gv.o graphml2gv.c
> rm lex.yy.c
> groff -Tps -man ./gc.1 | ps2pdf - - >gc.1.pdf
> cc1: some warnings being treated as errors
> make[4]: *** [Makefile:1062: gvgen.o] Error 1


The full build log is available from:
http://qa-logs.debian.net/2024/03/13/graphviz_2.42.2-8_unstable.log

All bugs filed during this archive rebuild are listed at:
https://bugs.debian.org/cgi-bin/pkgreport.cgi?tag=ftbfs-20240313;users=lu...@debian.org
or:
https://udd.debian.org/bugs/?release=na&merged=ign&fnewerval=7&flastmodval=7&fusertag=only&fusertagtag=ftbfs-20240313&fusertaguser=lu...@debian.org&allbugs=1&cseverity=1&ctags=1&caffected=1#results

A list of current common problems and possible solutions is available at
http://wiki.debian.org/qa.debian.org/FTBFS . You're welcome to contribute!

If you reassign this bug to another package, please mark it as 'affects'-ing
this package. See https://www.debian.org/Bugs/server-control#affects

If you fail to reproduce this, please provide a build log and diff it with mine
so that we can identify if something relevant changed in the meantime.

Reply via email to