[Galago-devel] org.freedesktop.Notifications GetServerInformation

[A. Walton]

On Tue, Sep 1, 2009 at 8:45 PM, Michael
Leibowitz<michael.leibowitz at intel.com> wrote:
> Hi,
>
> I've noticed an inconsistency in the prototype for GetServerInformation
> and what the spec reports.  The 0.9 version of the spec reports
> introspection XML as follows:
>    <method name="GetServerInformation">
>      <arg type="s" name="return_name" direction="out"/>
>      <arg type="s" name="return_vendor" direction="out"/>
>      <arg type="s" name="return_version" direction="out"/>
>    </method>
>
> However, the 0.3.xx versions of notification daemon has introspection
> XML as follows:
>    <method name="GetServerInformation">
>      <arg type="s" name="return_name" direction="out"/>
>      <arg type="s" name="return_vendor" direction="out"/>
>      <arg type="s" name="return_version" direction="out"/>
>      <arg type="s" name="return_spec_version" direction="out"/>
>    </method>

This is the correct version of the prototype. The above must be a typo
of some kind in that document. I don't know of any implementations
that support the former.

> Additionally, 0.3.xx versions report the spec version as 0.9.  This is
> not what is specified in the 0.9 specification, as seen above.
>
> The 0.4.0 version reports the specification version as 1.0, which
> doesn't appear at http://www.galago-project.org/specs/notification/
>
> I have to say that changing the prototype of GetServerInformation seems
> prone to problems.  If the client is adherent to the 0.9 or 0.7 version
> of the spec, then they are broken out of the box or they have to parse
> introspection information to figure which prototype of
> GetServerInformation to call.  This is, surely, not the desired effect.
>
> Perhaps somebody would like to make an amended 0.9 spec with the
> de-facto prototype?  Would it be possible to release the 1.0 spec?

There's a 0.10 version of the spec, which really needs to be moved to
FD.O but can't because the admins there are not willing to co-operate,
which contains the same 4-arg version as above.

-A. Walton

> Thanks
>
> --
> Michael Leibowitz <michael.leibowitz at intel.com>
>
> _______________________________________________
> galago-devel mailing list
> galago-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/galago-devel