Review Request 128335: Fix desktop files' Exec lines

Yoann Laissus yoann.laissus at gmail.com
Sat Jul 2 07:42:59 UTC 2016


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


Ship it!




Good catch, thanks for your patch !

- Yoann Laissus


On juil. 1, 2016, 12:03 après-midi, Wolfgang Bauer wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128335/
> -----------------------------------------------------------
> 
> (Updated juil. 1, 2016, 12:03 après-midi)
> 
> 
> Review request for KDE Utils and Yoann Laissus.
> 
> 
> Repository: kdf
> 
> 
> Description
> -------
> 
> The KF5 port uses QCommandLineParser which doesn't know the '-caption' parameter. The desktop files need to use '-qwindowtitle' instead.
> 
> 
> Diffs
> -----
> 
>   desktop/org.kde.kdf.desktop 43a720a 
>   desktop/org.kde.kwikdisk.desktop cf959e1 
> 
> Diff: https://git.reviewboard.kde.org/r/128335/diff/
> 
> 
> Testing
> -------
> 
> Run kdf and kwikdisk from the application launcher, they start fine now.
> 
> Before they didn't, and the message "Unknown options: c, a, p, t, i, o, n." appeared in .xsession-errors.
> 
> 
> Thanks,
> 
> Wolfgang Bauer
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-utils-devel/attachments/20160702/51c4112b/attachment.html>


More information about the Kde-utils-devel mailing list