Hello Neil,

Neil Williams schrieb am Tue 15. Aug, 19:36 (+0100):
> Jörg Sommer wrote:
> > Neil Williams schrieb am Mon 14. Aug, 14:38 (+0100):
> >> Jörg Sommer wrote:
> >>> Hello Neil,
> >>>> 2. -Wextra is *not supported* by gnucash and errors are to be expected.
> >>>> Don't use it.
> >>> -Wextra does not generate any errors. It warns about common cases where
> >>> errors can araise.
> >> -Wextra DOES generate errors under -Werror
> > 
> > Who enabled -Werror?
> 
> Upstream - sometime during the gnome2 transition.

No. That's not true. Look at
http://buildd.debian.org/fetch.php?&pkg=gnucash&ver=2.0.1-1&arch=powerpc&stamp=1155098349&file=log&as=raw

> One final comment: Check out the use of a more specific combination of
> -W options that highlight the real issues as compared to the pointless
> warnings. In particular, look at -Wsign-compare with -Wconversion
> *without* using -Wunused.

I did not enable -Wunused. That's a default option. Same source as above.

One final comment:
,----[ Debian policy 10.1 ]---
|      By default, when a package is being built, any binaries created should
|      include debugging information, as well as being compiled with
|      optimization.  You should also turn on as many reasonable compilation
|      warnings as possible; this makes life easier for porters, who can then
|      look at build logs for possible problems.  For the C programming
`----
And the same was suggested in the announcement of GCC 4.1 at beginning of
June.

BTW: You build without optimization. And you do not support
  DEB_BUILD_OPTIONS. Should I send a new bugreport?

Bye, Jörg.
-- 
[dpkg] We are the apt. Resistance is futile. You will be packaged.

Attachment: pgpusA3GLUuP8.pgp
Description: PGP signature

Reply via email to