On Sun, Jan 08 2023, Omar Polo <o...@omarpolo.com> wrote: > On 2023/01/08 16:25:16 +0100, Jérémie Courrèges-Anglas <j...@wxcvbn.org> > wrote: >> https://wxcvbn.org/~jca/build-failures/amd64-clang/2023-01-03/devel/fnc.txt > > fails building the bundled sqlite3 copy because of: > > : lib/sqlite3.c:57910:7: error: variable 'nUri' set but not used > [-Werror,-Wunused-but-set-variable] > : int nUri = 0; /* Number of URI parameters */ > : ^ > : 1 error generated. > > don't have the newer clang here yet but this should fix the build. > > ok?
I would prefer us to have a clear list of which port is broken by which warning before we start fixing them. I plan to go through the failures this evening. Here the port uses -Werror, and -Wunused-but-set-variable is not a new warning. I guess clang got better at this warning, I'm not sure we should expect much fallout from it and I think it's probably best to keep that one active as part of -Wall. But please wait until we have the bigger picture. Thanks. :) > Index: patches/patch-fnc_bld_mk > =================================================================== > RCS file: patches/patch-fnc_bld_mk > diff -N patches/patch-fnc_bld_mk > --- /dev/null 1 Jan 1970 00:00:00 -0000 > +++ patches/patch-fnc_bld_mk 8 Jan 2023 16:46:18 -0000 > @@ -0,0 +1,29 @@ > +drop -Werror > + > +Index: fnc.bld.mk > +--- fnc.bld.mk.orig > ++++ fnc.bld.mk > +@@ -11,7 +11,7 @@ HASH != cut -f 1 manifest.uuid > + DATE != sed '2q;d' manifest | cut -d ' ' -f 2 | tr T ' ' > + > + # FLAGS NEEDED TO BUILD SQLITE3 > +-SQLITE_CFLAGS = ${CFLAGS} -Wall -Werror -Wno-sign-compare -pedantic > -std=c99 \ > ++SQLITE_CFLAGS = ${CFLAGS} -Wall -Wno-sign-compare -pedantic -std=c99 \ > + -DNDEBUG=1 \ > + -DSQLITE_DQS=0 \ > + -DSQLITE_DEFAULT_MEMSTATUS=0 \ > +@@ -37,12 +37,12 @@ SQLITE_CFLAGS = ${CFLAGS} -Wall -Werror -Wno-sign-comp > + -DSQLITE_TRUSTED_SCHEMA=0 > + > + # FLAGS NEEDED TO BUILD LIBFOSSIL > +-FOSSIL_CFLAGS = ${CFLAGS} -Wall -Werror -Wsign-compare -pedantic > -std=c99 > ++FOSSIL_CFLAGS = ${CFLAGS} -Wall -Wsign-compare -pedantic -std=c99 > + > + # On SOME Linux (e.g., Ubuntu 18.04.6), we have to include wchar curses from > + # I/.../ncursesw, but linking to -lncursesw (w/ no special -L path) works > fine. > + # FLAGS NEEDED TO BUILD FNC > +-FNC_CFLAGS = ${CFLAGS} -Wall -Werror -Wsign-compare -pedantic > -std=c99 \ > ++FNC_CFLAGS = ${CFLAGS} -Wall -Wsign-compare -pedantic -std=c99 \ > + -I./lib -I./include -I/usr/include/ncursesw \ > + -D_XOPEN_SOURCE_EXTENDED -DVERSION=${VERSION} -DHASH=${HASH} \ > + -DDATE="${DATE}" > -- jca | PGP : 0x1524E7EE / 5135 92C1 AD36 5293 2BDF DDCC 0DFA 74AE 1524 E7EE
signature.asc
Description: PGP signature