On Wednesday 01 December 2010, Tom Callaway wrote: [...] > https://fedoraproject.org/wiki/TomCallaway/Systemd_Revised_Draft [...] > Please look it over and give any feedback. I haven't educated myself much at all about systemd, and didn't read the draft completely, but here are some comments: | BusName= should be set for all services connecting to D-Bus. (i.e. it is a | must for those where Type=dbus, but might make sense otherwise, too) Omit | this option if your service does not take a name on the bus. Should the "the bus" at the end be replaced by "D-Bus" or "a bus" or "any bus" or "the bus specified with Type=..." or...? Not sure exactly what bus does it mean. | ExecReload= should be specified for all services supporting reload. It is | highly recommended to add code here that synchronously reloads the | configuration file here (i.e. /bin/kill -HUP $MAINPID is usually a poor | choice, due to its asynchronous nature). Omit this option if your service | does not support reloading. Does this mean that services that do support reloading but only asynchronously should not include this option? I guess not, i.e. asynch reload is better than no reload? If that's correct, explicitly mentioning that would be an improvement. -- packaging mailing list packaging@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/packaging
