[Fedora-packaging] Revised systemd Guidelines

[Tom Callaway]

On 12/02/2010 05:04 PM, Ville Skyttä wrote:
> | 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.

Should probably be replaced by "any bus". Lennart, do you agree?

> | 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.

I'll defer to Lennart here as to what he thinks is acceptable.

~tom

==
Fedora Project