Re: Review Request 117087: co-installability: rename entry.desktop to kf5_entry.desktop

2014-03-28 Thread Jonathan Riddell

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

(Updated March 28, 2014, 4:16 p.m.)


Status
--

This change has been marked as submitted.


Review request for KDE Frameworks and Chusslove Illich.


Repository: kde4support


Description
---

stop clashing files on install with kdelibs4 and rename entry.desktop to 
kf5_entry.desktop


Diffs
-

  src/CMakeLists.txt 307cafa 
  src/kdecore/klocale_kde.cpp d1bedee 

Diff: https://git.reviewboard.kde.org/r/117087/diff/


Testing
---


Thanks,

Jonathan Riddell

___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel


Re: Review Request 117087: co-installability: rename entry.desktop to kf5_entry.desktop

2014-03-28 Thread Commit Hook

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


This review has been submitted with commit 
a33377b84b9b312cf5f1f2baae42e6e514731d64 by Jonathan Riddell to branch master.

- Commit Hook


On March 26, 2014, 1:44 p.m., Jonathan Riddell wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://git.reviewboard.kde.org/r/117087/
 ---
 
 (Updated March 26, 2014, 1:44 p.m.)
 
 
 Review request for KDE Frameworks and Chusslove Illich.
 
 
 Repository: kde4support
 
 
 Description
 ---
 
 stop clashing files on install with kdelibs4 and rename entry.desktop to 
 kf5_entry.desktop
 
 
 Diffs
 -
 
   src/CMakeLists.txt 307cafa 
   src/kdecore/klocale_kde.cpp d1bedee 
 
 Diff: https://git.reviewboard.kde.org/r/117087/diff/
 
 
 Testing
 ---
 
 
 Thanks,
 
 Jonathan Riddell
 


___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel


Re: Review Request 117087: co-installability: rename entry.desktop to kf5_entry.desktop

2014-03-28 Thread Aleix Pol Gonzalez


 On March 28, 2014, 4:16 p.m., Commit Hook wrote:
  This review has been submitted with commit 
  a33377b84b9b312cf5f1f2baae42e6e514731d64 by Jonathan Riddell to branch 
  master.

Why did you ignore my comment? This breaks entry.desktop files usage.


- Aleix


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


On March 28, 2014, 4:16 p.m., Jonathan Riddell wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://git.reviewboard.kde.org/r/117087/
 ---
 
 (Updated March 28, 2014, 4:16 p.m.)
 
 
 Review request for KDE Frameworks and Chusslove Illich.
 
 
 Repository: kde4support
 
 
 Description
 ---
 
 stop clashing files on install with kdelibs4 and rename entry.desktop to 
 kf5_entry.desktop
 
 
 Diffs
 -
 
   src/CMakeLists.txt 307cafa 
   src/kdecore/klocale_kde.cpp d1bedee 
 
 Diff: https://git.reviewboard.kde.org/r/117087/diff/
 
 
 Testing
 ---
 
 
 Thanks,
 
 Jonathan Riddell
 


___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel


Re: Review Request 117087: co-installability: rename entry.desktop to kf5_entry.desktop

2014-03-28 Thread Jonathan Riddell


 On March 28, 2014, 4:16 p.m., Commit Hook wrote:
  This review has been submitted with commit 
  a33377b84b9b312cf5f1f2baae42e6e514731d64 by Jonathan Riddell to branch 
  master.
 
 Aleix Pol Gonzalez wrote:
 Why did you ignore my comment? This breaks entry.desktop files usage.

Sorry I was wanting to get it in for the KF5 beta, I'll post a patch for 
kde-runtime shortly, anything else I should be updating?


- Jonathan


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


On March 28, 2014, 4:16 p.m., Jonathan Riddell wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://git.reviewboard.kde.org/r/117087/
 ---
 
 (Updated March 28, 2014, 4:16 p.m.)
 
 
 Review request for KDE Frameworks and Chusslove Illich.
 
 
 Repository: kde4support
 
 
 Description
 ---
 
 stop clashing files on install with kdelibs4 and rename entry.desktop to 
 kf5_entry.desktop
 
 
 Diffs
 -
 
   src/CMakeLists.txt 307cafa 
   src/kdecore/klocale_kde.cpp d1bedee 
 
 Diff: https://git.reviewboard.kde.org/r/117087/diff/
 
 
 Testing
 ---
 
 
 Thanks,
 
 Jonathan Riddell
 


___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel


Re: Review Request 117087: co-installability: rename entry.desktop to kf5_entry.desktop

2014-03-28 Thread Aleix Pol Gonzalez


 On March 28, 2014, 4:16 p.m., Commit Hook wrote:
  This review has been submitted with commit 
  a33377b84b9b312cf5f1f2baae42e6e514731d64 by Jonathan Riddell to branch 
  master.
 
 Aleix Pol Gonzalez wrote:
 Why did you ignore my comment? This breaks entry.desktop files usage.
 
 Jonathan Riddell wrote:
 Sorry I was wanting to get it in for the KF5 beta, I'll post a patch for 
 kde-runtime shortly, anything else I should be updating?


There's a list of uses of these files in this thread:
https://www.mail-archive.com/kde-frameworks-devel@kde.org/msg11839.html


- Aleix


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


On March 28, 2014, 4:16 p.m., Jonathan Riddell wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://git.reviewboard.kde.org/r/117087/
 ---
 
 (Updated March 28, 2014, 4:16 p.m.)
 
 
 Review request for KDE Frameworks and Chusslove Illich.
 
 
 Repository: kde4support
 
 
 Description
 ---
 
 stop clashing files on install with kdelibs4 and rename entry.desktop to 
 kf5_entry.desktop
 
 
 Diffs
 -
 
   src/CMakeLists.txt 307cafa 
   src/kdecore/klocale_kde.cpp d1bedee 
 
 Diff: https://git.reviewboard.kde.org/r/117087/diff/
 
 
 Testing
 ---
 
 
 Thanks,
 
 Jonathan Riddell
 


___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel


Re: Review Request 117087: co-installability: rename entry.desktop to kf5_entry.desktop

2014-03-27 Thread Kevin Ottens

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

Ship it!


Ship It!

- Kevin Ottens


On March 26, 2014, 1:44 p.m., Jonathan Riddell wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://git.reviewboard.kde.org/r/117087/
 ---
 
 (Updated March 26, 2014, 1:44 p.m.)
 
 
 Review request for KDE Frameworks and Chusslove Illich.
 
 
 Repository: kde4support
 
 
 Description
 ---
 
 stop clashing files on install with kdelibs4 and rename entry.desktop to 
 kf5_entry.desktop
 
 
 Diffs
 -
 
   src/CMakeLists.txt 307cafa 
   src/kdecore/klocale_kde.cpp d1bedee 
 
 Diff: https://git.reviewboard.kde.org/r/117087/diff/
 
 
 Testing
 ---
 
 
 Thanks,
 
 Jonathan Riddell
 


___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel


Re: Review Request 117087: co-installability: rename entry.desktop to kf5_entry.desktop

2014-03-27 Thread Aleix Pol Gonzalez

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


Again, these will go to kde4support, calling them kf5_ is weird...

Additionally, these are used in many places. These should be updated as well.

- Aleix Pol Gonzalez


On March 26, 2014, 1:44 p.m., Jonathan Riddell wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://git.reviewboard.kde.org/r/117087/
 ---
 
 (Updated March 26, 2014, 1:44 p.m.)
 
 
 Review request for KDE Frameworks and Chusslove Illich.
 
 
 Repository: kde4support
 
 
 Description
 ---
 
 stop clashing files on install with kdelibs4 and rename entry.desktop to 
 kf5_entry.desktop
 
 
 Diffs
 -
 
   src/CMakeLists.txt 307cafa 
   src/kdecore/klocale_kde.cpp d1bedee 
 
 Diff: https://git.reviewboard.kde.org/r/117087/diff/
 
 
 Testing
 ---
 
 
 Thanks,
 
 Jonathan Riddell
 


___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel