On Mon, Jan 10, 2022 at 10:55:54PM +0100, Jeremie Courreges-Anglas wrote:
> On Mon, Jan 10 2022, Alexander Bluhm <alexander.bl...@gmx.net> wrote:
> > On Sat, Jan 01, 2022 at 10:52:37AM +0100, Jeremie Courreges-Anglas wrote:
> >> On Sat, Jan 01 2022, Jeremie Courreges-Anglas <j...@wxcvbn.org> wrote:
> >> > On Fri, Dec 31 2021, Stefan Hagen <sh+openbsd-po...@codevoid.de> wrote:
> >> >> Christian Weisgerber wrote:
> >> >>> Here's a reminder that these ports still fail to build since the
> >> >>> LLVM 13 update:
> >> >>> 
> >> >>>   devel/qbs
> >> >>>   lang/libv8
> >> >>>   misc/posixtestsuite
> >> >>
> >> >> Here -Werror is set and some variables now throw an -Wunused-but-set 
> >> >> error.
> >> >> This happens frequently in this testsuite.
> >> >>
> >> >> Shall we switch it off?
> >> >
> >> > This is a clang-only flag, you can't use it with base-gcc.  Here's a way
> >> > to handle this.
> >> 
> >> That would work if CFLAGS was respected, but it isn't...
> >
> > Who needs this -Werror anyway?  Without it, it manages to compile
> > more tests.
> >
> > ok?
> 
> Yep.
> 
> But still it would be better to address the "does not respect CFLAGS" part.
> Can be done in a subsequent commit.

Seems to work as my patch contains a CFLAGS +=

bluhm

Reply via email to