Signed-off-by: Fabiano Fidêncio fidencio@redhat.com
rebased
You need to prefix each line with a "* " to create a list
..."on platforms"..
s/tipically/typically/
s/soolution/solution/
..."will communicate"..
s/unis/unix/
s/nss_ss/nss_sss/
If you're changing the word, would you also marking it as non-proportional?
sssd-pam-private.socket
we also have secrets and KCM :)
on the monitor side
It might be worth mentioning (and maybe adding an example with a single service) that the socket and the service might need to be enabled.
Apart from the typos and nitpicks above LGTM.
BTW ifp is not socket activated service :-) I think the name of design page should be systemd activatable responders.
I do mention it in the page. Do you really think changing the page's name is necessary? (I can do if you say so ...)
Yes, please.
It is not just about page(file) name. But the title of page also says Socket Activatable Responders.
Upstart can also start socket activated services. http://upstart.ubuntu.com/cookbook/#id277 Therefore we need to be explicit about systemd activated services.
I merged PR#7 so now there is a conflict..
@lslebodn patchset updated addressing your comments and also rebased in order to avoid the conflict.
Thank you for changes. I let Jakub to have last word. (ACK/merge it ... :-)
There are two warnings when building the HTML documentation: /home/remote/jhrozek/devel/sssd-docs/design_pages/systemd_activatable_responders.rst:4: WARNING: Title underline too short.
/home/remote/jhrozek/devel/sssd-docs/design_pages/systemd_activatable_responders.rst:247: ERROR: Unexpected indentation.
In the second one, you need to add a blank line between the text and the start of the unordered list.
Pushed as 12af9f86761137c9bc20af8106e5643ab3193e9a
Pull-Request has been closed by jhrozek
Signed-off-by: Fabiano Fidêncio fidencio@redhat.com