Bug 86162

Summary:	[SPEC UPDATE] please document that <annotation> is allowed within <arg> in the XML introspection data
Product:	dbus	Reporter:	Lennart Poettering <lennart>
Component:	core	Assignee:	D-Bus Maintainers <dbus>
Status:	RESOLVED FIXED	QA Contact:	D-Bus Maintainers <dbus>
Severity:	normal
Priority:	medium	CC:	bugzilla
Version:	unspecified
Hardware:	Other
OS:	All
Whiteboard:
i915 platform:		i915 features:
Attachments:	doc: Add missing space in introspect.xsl spec: Allow <annotation> in <arg> elements in introspection XML

Description Lennart Poettering 2014-11-11 16:06:54 UTC

As discussed on the ML: the spec should allow embedding <annotation> within <arg>. Apparently both gdbus and qt make use of this, and it probably makes sense to allow this, hence this should be mentioned in the spec.

Also, the DTD shipped with dbus should really allows this too, since it is the only formalized description of the XML format we have.

See this thread for more info:

http://lists.freedesktop.org/archives/dbus/2014-November/016399.html

Comment 1 Philip Withnall 2016-10-01 11:49:48 UTC

Created attachment 126932 [details] [review]
doc: Add missing space in introspect.xsl

This adds a space in the output between ‘annotation’ and the key of the annotation. A Saturday afternoon cosmetic fix.

Comment 2 Philip Withnall 2016-10-01 11:50:17 UTC

Created attachment 126933 [details] [review]
spec: Allow <annotation> in <arg> elements in introspection XML

This is widely used in practice (especially by GLib — just look at files in /usr/share/dbus-1/interfaces/), and there is no reason not to allow it. Update the specification, introspection DTD and XSL file to allow and represent it.

Comment 3 Simon McVittie 2016-10-03 10:03:30 UTC

Comment on attachment 126932 [details] [review]
doc: Add missing space in introspect.xsl

Review of attachment 126932 [details] [review]:
-----------------------------------------------------------------

XSLT! I'd forgotten we had any of this.

(Looks fine.)

Comment 4 Simon McVittie 2016-10-03 10:04:22 UTC

Comment on attachment 126933 [details] [review]
spec: Allow <annotation> in <arg> elements in introspection XML

Review of attachment 126933 [details] [review]:
-----------------------------------------------------------------

I assume if any maintainers were going to object to this they would have done so on Lennart's bug report. Let's do this.

Comment 5 Simon McVittie 2016-10-03 10:46:06 UTC

Fixed in git for 1.11.6, thanks.

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.