On Tue, Dec 07 2021, "Gonzalo L. Rodriguez" <gonz...@x61.sh> wrote: > On Mon, 06 Dec 2021 at 23:43:12 +0100, Jeremie Courreges-Anglas wrote: >> On Sun, Dec 05 2021, Christian Weisgerber <na...@mips.inka.de> wrote: >> > I ran another amd64 bulk build with base clang updated to LLVM 13 >> > and --allow-shlib-undefined as the default again (~patrick/llvm.v6). >> > I also put in a tentative fix for security/nss. >> > >> > Failure logs: >> > http://build-failures.rhaalovely.net/amd64-clang/2021-12-04/ >> > >> > Triage: >> >> > emulators/nono -Werror,-Wunused-but-set-variable >> >> Looks as simple as dropping -Werror. ok? >> >> >> Index: Makefile >> =================================================================== >> RCS file: /cvs/ports/emulators/nono/Makefile,v >> retrieving revision 1.17 >> diff -u -p -r1.17 Makefile >> --- Makefile 5 Oct 2021 10:15:59 -0000 1.17 >> +++ Makefile 6 Dec 2021 22:40:01 -0000 >> @@ -30,6 +30,9 @@ LIB_DEPENDS= x11/wxWidgets >> NO_TEST= Yes >> CONFIGURE_STYLE= gnu >> >> +post-extract: >> + sed -i 's/-Werror//g' ${WRKSRC}/Makefile.inc >> + >> post-install: >> ${INSTALL_DATA_DIR} ${PREFIX}/share/doc/nono >> ${INSTALL_DATA} ${WRKSRC}/doc/index.html >> ${PREFIX}/share/doc/nono/manual.html >> >> -- >> jca | PGP : 0x1524E7EE / 5135 92C1 AD36 5293 2BDF DDCC 0DFA 74AE 1524 E7EE >> > > Whoops I didn't see this one, how about with the update included?
I can confirm nono-0.2.4 builds with this diff (it still errors out if -Werror is left in place). > OK? LGTM ports-wise, ok jca@ PS: in case you don't intend to commit the update to 0.2.4 very soon, we should fix the llvm 13 error first. -- jca | PGP : 0x1524E7EE / 5135 92C1 AD36 5293 2BDF DDCC 0DFA 74AE 1524 E7EE
