Review Request: updated kded4 man page

[Jaime Torres]

> On 2010-10-08 18:56:28, Arno Rehn wrote:
> > /trunk/KDE/kdelibs/doc/kded4/man-kded4.8.docbook, line 104
> > <http://svn.reviewboard.kde.org/r/5557/diff/1/?file=39102#file39102line104>
> >
> >     I think this should be 'Provides'. Maybe rephrase, I at least don't quite get what it means.

I think I must practice with Ktouch in English. :-(


> On 2010-10-08 18:56:28, Arno Rehn wrote:
> > /trunk/KDE/kdelibs/doc/kded4/man-kded4.8.docbook, line 146
> > <http://svn.reviewboard.kde.org/r/5557/diff/1/?file=39102#file39102line146>
> >
> >     'kio slave' is probably the correct term here, not 'protocol'. I don't know how technical we want to get here, though.

Just typed from the description in system settings. Changed here.


> On 2010-10-08 18:56:28, Arno Rehn wrote:
> > /trunk/KDE/kdelibs/doc/kded4/man-kded4.8.docbook, line 159
> > <http://svn.reviewboard.kde.org/r/5557/diff/1/?file=39102#file39102line159>
> >
> >     s/provices/provides/
> >     s/notification to/notifications for/

yes, ktouch to the rescue. :-)


> On 2010-10-08 18:56:28, Arno Rehn wrote:
> > /trunk/KDE/kdelibs/doc/kded4/man-kded4.8.docbook, line 205
> > <http://svn.reviewboard.kde.org/r/5557/diff/1/?file=39102#file39102line205>
> >
> >     Clarify, this sounds a bit strange.

I also do not know what it does. It is typed as seen in the system settings.


> On 2010-10-08 18:56:28, Arno Rehn wrote:
> > /trunk/KDE/kdelibs/doc/kded4/man-kded4.8.docbook, line 255
> > <http://svn.reviewboard.kde.org/r/5557/diff/1/?file=39102#file39102line255>
> >
> >     Maybe 'unmounts' instead of 'releases' ?
> >     Again, don't know how technical we want to get in the manpages.

I think it does both tasks. The eject button is managed by software in some SCSI disks (I have one of those) and some DVD. 


- Jaime


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


On 2010-10-08 21:33:00, Jaime Torres wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://svn.reviewboard.kde.org/r/5557/
> -----------------------------------------------------------
> 
> (Updated 2010-10-08 21:33:00)
> 
> 
> Review request for kdelibs.
> 
> 
> 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
> -----
> 
>   /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/20101008/de29d878/attachment.htm>