Review Request: [dolphin] Fix the warning generated by desktop-file-validate for dolphin.desktop

2012-06-18 Thread Jekyll Wu

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

Review request for KDE Base Apps and Peter Penz.


Description
---

desktop-file-validate dolphin.dekstop generates this:

dolphin.desktop: error: (will be fatal in the future): value FileManager in 
key Categories in group Desktop Entry requires another category to be 
present among the following categories: System;FileTools

FDO menu specification[1] suggests System;FileTools should be used together 
with the FileManager category.

The patch simply adds the missing FileTools

[1] http://standards.freedesktop.org/menu-spec/latest/apa.html


Diffs
-

  dolphin/src/dolphin.desktop cd9da17 

Diff: http://git.reviewboard.kde.org/r/105288/diff/


Testing
---


Thanks,

Jekyll Wu



Re: Review Request: [dolphin] Fix the warning generated by desktop-file-validate for dolphin.desktop

2012-06-18 Thread Peter Penz

---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/105288/#review14833
---

Ship it!


Thanks, looks fine! (btw: from my point of view no review-request would have 
been necessary for this fix, but of course it is exemplary doing it this way 
:-))

- Peter Penz


On June 18, 2012, 1:47 a.m., Jekyll Wu wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 http://git.reviewboard.kde.org/r/105288/
 ---
 
 (Updated June 18, 2012, 1:47 a.m.)
 
 
 Review request for KDE Base Apps and Peter Penz.
 
 
 Description
 ---
 
 desktop-file-validate dolphin.dekstop generates this:
 
 dolphin.desktop: error: (will be fatal in the future): value FileManager in 
 key Categories in group Desktop Entry requires another category to be 
 present among the following categories: System;FileTools
 
 FDO menu specification[1] suggests System;FileTools should be used together 
 with the FileManager category.
 
 The patch simply adds the missing FileTools
 
 [1] http://standards.freedesktop.org/menu-spec/latest/apa.html
 
 
 Diffs
 -
 
   dolphin/src/dolphin.desktop cd9da17 
 
 Diff: http://git.reviewboard.kde.org/r/105288/diff/
 
 
 Testing
 ---
 
 
 Thanks,
 
 Jekyll Wu
 




Re: Review Request: [dolphin] Fix the warning generated by desktop-file-validate for dolphin.desktop

2012-06-18 Thread Commit Hook

---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/105288/#review14836
---


This review has been submitted with commit 
a7d999b7ba3e35069649eb434e9c4e88052a25d0 by Jekyll Wu to branch master.

- Commit Hook


On June 18, 2012, 1:47 a.m., Jekyll Wu wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 http://git.reviewboard.kde.org/r/105288/
 ---
 
 (Updated June 18, 2012, 1:47 a.m.)
 
 
 Review request for KDE Base Apps and Peter Penz.
 
 
 Description
 ---
 
 desktop-file-validate dolphin.dekstop generates this:
 
 dolphin.desktop: error: (will be fatal in the future): value FileManager in 
 key Categories in group Desktop Entry requires another category to be 
 present among the following categories: System;FileTools
 
 FDO menu specification[1] suggests System;FileTools should be used together 
 with the FileManager category.
 
 The patch simply adds the missing FileTools
 
 [1] http://standards.freedesktop.org/menu-spec/latest/apa.html
 
 
 Diffs
 -
 
   dolphin/src/dolphin.desktop cd9da17 
 
 Diff: http://git.reviewboard.kde.org/r/105288/diff/
 
 
 Testing
 ---
 
 
 Thanks,
 
 Jekyll Wu