Bug 66481

Summary: [PATCH] DBus Spec: fix sample service file
Product: dbus Reporter: Chengwei Yang <chengwei.yang.cn>
Component: coreAssignee: Havoc Pennington <hp>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: medium CC: chengwei.yang.cn
Version: 1.5Keywords: patch
Hardware: Other   
OS: All   
Whiteboard:
i915 platform: i915 features:
Attachments: [PATCH] DBus Spec: Fix sample service file (merged)
[PATCH] DBus Spec: fix examples namespace to com.example

Description Chengwei Yang 2013-07-02 05:34:16 UTC
Figure 9. Example service description file

            # Sample service description file
            [D-BUS Service]
            Names=org.freedesktop.ConfigurationDatabase;org.gnome.GConf;
            Exec=/usr/libexec/gconfd-2
          
There are two errors in the above sample service file.
First, a typo, Names should be Name.
Second, can not specify multiple names.
Comment 1 Chengwei Yang 2013-07-02 05:35:45 UTC
Created attachment 81842 [details] [review]
[PATCH] DBus Spec: Fix sample service file (merged)
Comment 2 Simon McVittie 2013-08-22 18:13:28 UTC
Comment on attachment 81842 [details] [review]
[PATCH] DBus Spec: Fix sample service file (merged)

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

Yes.

We ought to change this to be in example.com's namespace too, really - too many people use names in freedesktop.org that they don't actually control, because they've cargo-culted "D-Bus names start with org.freedesktop".
Comment 3 Chengwei Yang 2013-08-23 07:42:20 UTC
Yeah, someone did confused. http://lists.freedesktop.org/archives/dbus/2008-January/009113.html

I'll upload a new patch to fix samples to com.example or /com/example from org.freedesktop or /org/freedesktop.
Comment 4 Chengwei Yang 2013-08-23 07:56:01 UTC
Created attachment 84501 [details] [review]
[PATCH] DBus Spec: fix examples namespace to com.example
Comment 5 Simon McVittie 2013-08-23 09:04:50 UTC
Comment on attachment 84501 [details] [review]
[PATCH] DBus Spec: fix examples namespace to com.example

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

Ideal.
Comment 6 Simon McVittie 2013-08-23 10:47:54 UTC
Fixed in git for 1.7.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.