Review Request: updated kded4 man page

[Jaime Torres]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/5557/
-----------------------------------------------------------

(Updated 2010-10-11 12:11:36.026257)


Review request for kdelibs.


Changes
-------

Done David suggestions, after a bugday.
Also introduced how to enable/disable the init kded4 modules.
Is it ok that way of describing the menus?
If yes, probably it is ready for shipping.


Summary
-------

Three questions:
* Is it wrote in correct English?
* Is it ok to list the modules in that way and in that page?
* In the KCM module Service Manager (from where I've copied the names and descriptions) there are 5 descriptions missing (also in the man page):
  * KDED Subversion module
  * KDED module for screen-managment
  * Keyboard Daemon
  * Display Managment change monitor
  * K Remote Control Daemon
  Please, non trivial descriptions?


This addresses bug 242582.
    https://bugs.kde.org/show_bug.cgi?id=242582


Diffs (updated)
-----

  /trunk/KDE/kdelibs/doc/kded4/man-kded4.8.docbook 1183954 

Diff: http://svn.reviewboard.kde.org/r/5557/diff


Testing
-------

The man page is more complete, and now can search for out of space


Thanks,

Jaime

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20101011/0bb46a1d/attachment.htm>