[pacman-dev] [PATCH] meson: use 'pedantic' compiler warning level
eschwartz at archlinux.org
Fri Mar 27 03:36:25 UTC 2020
On 3/26/20 4:42 PM, Eli Schwartz wrote:
> On 3/26/20 3:32 PM, Anatol Pomozov wrote:
>> In this case I would like to propose to move forward with my original
>> patch and enable 'pedantic' warning level by default. It will match
>> the flag with the Makefile based build.
> The original patch did not clarify this was about parity with autotools.
> I've looked around a bit and my conclusions are as follows:
> - We always defaulted to -pedantic -D_GNU_SOURCE -Wall in autotools,
> plus additional flags if --enable-warningflags
> - meson will, with warning_level 3, add -Wall -Wextra -Wpedantic, which
> means this patch would add the heretofore-unseen -Wextra
> (warning_level 1 only adds -Wall, 2 adds -Wall -Wextra, no option is
> available for those who like -Wpedantic but don't want -Wextra)
> Since autotools unconditionally added these flags, parity suggests
> always adding them in meson too. Do we care about -Wextra? We currently
> build with -Wextra without warnings being emitted, anyway.
Allan confirmed on IRC that we do care, and this patch will not be
accepted because we do not want -Wextra.
> If -Wextra is fine then we can just take this current patch, but amend
> it to add -Wextra to configure.ac near
> if test "x$debug" = "xyes" ; then
> Anatol, can you also update the commit message to mention that this is
> implementing parity with the default autotools warning level?
Bug Wrangler and Trusted User
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 1601 bytes
Desc: OpenPGP digital signature
More information about the pacman-dev