[packagekit] Werror and maintainer-mode

Mounir Lamouri mounir.lamouri at gmail.com
Sat Jul 18 01:40:32 PDT 2009


Hi,

I think Werror should be removed or be optional in packagekit. You can see the
reasons here [1].
We talked about that quickly a few weeks ago and the idea that showed up was to
enable it with maintainer-mode enabled. Actually, maintainer-mode is not
available in packagekit configure script (it should be removed from autogen by
the way) and maintainer-mode shouldn't be used for that [2].

[1] http://blog.flameeyes.eu/2009/02/25/future-proof-your-code-dont-use-werror
[2] http://blog.flameeyes.eu/2008/06/13/maintaner-mode

I see two solutions:
- automatically add -Werror when --enable-local is set.
- add a new option --enable-strict which will enable this flag.

Thanks,
Mounir



More information about the PackageKit mailing list