Review Request 115063: Created missing manpage for desktoptojson binary

Sebastian Kügler sebas at kde.org
Fri Jan 17 10:46:30 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/115063/#review47561
-----------------------------------------------------------


Hi Scarlett,

Congrats to your first patch (that I see coming by)! I think it's a useful contribution and as far as I can judge, almost good to go in. I've put a comment inline how the explanation could be further improved.

I don't know much about manpages, so I'll let someone else comment on the mechanics.

Thanks for taking care of this.


docs/desktoptojson/man-desktoptojson.8.docbook
<https://git.reviewboard.kde.org/r/115063/#comment33781>

    A link to the API docs for KPluginTrader would be useful. There, the user (developer, really, this is a development tool) finds more extensive documentation about its usage. As API documentation location hasn't really settled down yet, adding a sentence like this would probably be enough already:
    
    "For further reference how to use this programme and its underlying mechanism, please refer to the documentation for the KPluginTrader class in KDE's KService framework."


- Sebastian Kügler


On Jan. 17, 2014, 2:52 a.m., Scarlett Clark wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/115063/
> -----------------------------------------------------------
> 
> (Updated Jan. 17, 2014, 2:52 a.m.)
> 
> 
> Review request for Documentation, KDE Frameworks, Jonathan Riddell, and Sebastian Kügler.
> 
> 
> Repository: kservice
> 
> 
> Description
> -------
> 
> While building the package for kubuntu lintian complained about missing manpage for binary. So I created one :) This is my first manpage so be kind. I have no idea how to determine what type eg 1, 7, 8.
> 
> 
> Diffs
> -----
> 
>   docs/CMakeLists.txt 86ed60b 
>   docs/desktoptojson/CMakeLists.txt PRE-CREATION 
>   docs/desktoptojson/man-desktoptojson.8.docbook PRE-CREATION 
> 
> Diff: https://git.reviewboard.kde.org/r/115063/diff/
> 
> 
> Testing
> -------
> 
>  meinproc5 --stylesheet /usr/share/ksgmltools2/customization/kde-include-man.xsl --check man-desktoptojson.8.docbook 
> 
> 
> Thanks,
> 
> Scarlett Clark
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140117/1bc29268/attachment.html>


More information about the Kde-frameworks-devel mailing list