[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Help with -Werror
From: |
Russ Allbery |
Subject: |
Re: Help with -Werror |
Date: |
Wed, 24 Apr 2019 22:02:54 -0700 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux) |
Bob Friesenhahn <address@hidden> writes:
> One possibility would be to add configure support for
> --enable-maintainer-mode and then change the defaults when it is
> enabled. Another possiblity is to add a configure option to enable the
> option, which developers should always enable.
I personally use a separate target (make warnings), which builds both the
source and the tests with various warning flags enabled and -Werror.
# Separate target for a human to request building everything with as many
# compiler warnings enabled as possible.
warnings:
$(MAKE) V=0 CFLAGS='$(WARNINGS_CFLAGS) $(AM_CFLAGS)'
$(MAKE) V=0 CFLAGS='$(WARNINGS_CFLAGS) $(AM_CFLAGS)' $(check_PROGRAMS)
As a developer, I don't really like the solutions that require I muck
around with configure flags, because configure takes quite a while to run
(much longer than make clean && make warnings for my typical small
projects) and I like to be able to do regular builds while I'm getting
something working or iterating on some test and then a warnings build once
the code basically works.
Completely agreed that -Werror is an inappropriate default. It's a
development tool, and is just going to cause frustration for someone who
has downloaded the package and wants to use it, not develop on it.
--
Russ Allbery (address@hidden) <http://www.eyrie.org/~eagle/>