Bug 9754

Summary:	DBUS-PYTHON blocker warnings
Product:	dbus	Reporter:	Eduardo Silva <edsiper>
Component:	python	Assignee:	John (J5) Palmieri <johnp>
Status:	RESOLVED FIXED	QA Contact:
Severity:	blocker
Priority:	highest
Version:	unspecified
Hardware:	Other
OS:	Linux (All)
Whiteboard:
i915 platform:		i915 features:

Description Eduardo Silva 2007-01-24 11:48:47 UTC

Trying to compile Dbus-Python (GIT):
Commit: 4c9b10a511f9465ee87e4f87065445c04a34d7e5

Error:

abstract.c:45: warning: missing initializer
abstract.c:45: warning: (near initialization for `DBusPythonInt_tp_members[1].type')
abstract.c:139: warning: missing initializer
abstract.c:139: warning: (near initialization for `DBusPyIntBase_Type.tp_free')
abstract.c:156: warning: missing initializer
abstract.c:156: warning: (near initialization for `DBusPythonFloat_tp_members[1].type')
abstract.c:250: warning: missing initializer
abstract.c:250: warning: (near initialization for `DBusPyFloatBase_Type.tp_free')
abstract.c:358: warning: missing initializer
abstract.c:358: warning: (near initialization for `DBusPyStrBase_Type.tp_free')
abstract.c:466: warning: missing initializer

Comment 1 Simon McVittie 2007-01-24 12:28:31 UTC

These warnings are not a problem, which is why configure.ac tries to disable them by using -Wno-missing-field-initializers; this is how you do multiple-version compatibility in CPython, which has class structures that increase in size every few versions. The CPython API/ABI is such that leaving the end of the structure implicitly zero-initialized is always safe.

What gcc version are you using, and does it support the -Wno-missing-field-initializers option? We might have to do something like not using -Wextra unless -Wno-missing-field-initializers also works.

In a release tarball warnings aren't fatal in any case, -Werror is only enabled for the git tree.

Comment 2 Marco Pesenti Gritti 2007-01-24 12:44:49 UTC

Could Werror be enabled only for maintainer mode or something... This is going to screw us pretty badly.

Comment 3 Simon McVittie 2007-01-30 07:30:12 UTC

Changed in git as requested. (Doesn't need to go into a release to be relevant, since -Werror only happens in non-release builds)

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.