Re: Review Request 122313: Expose to world whether KPty has been built with utempter library

2015-03-19 Thread David Faure

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

Ship it!


- David Faure


On Jan. 29, 2015, 6:54 p.m., Hrvoje Senjan wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://git.reviewboard.kde.org/r/122313/
 ---
 
 (Updated Jan. 29, 2015, 6:54 p.m.)
 
 
 Review request for KDE Frameworks and David Faure.
 
 
 Repository: kpty
 
 
 Description
 ---
 
 Equivalent to https://svn.reviewboard.kde.org/r/2468/
 It was lost in KF5 porting, and it was directly used to determine whether 
 kwrited should be built as module, or executable (in this case, it would be a 
 SUID binary, which Qt5 no longer -by default- allows)
 
 
 Diffs
 -
 
   CMakeLists.txt 7fe77da7b0bc97c6f64db4fcc63ef7831fa065b1 
   KF5PtyConfig.cmake.in 04bde7bffd209b57e755a66278025ee8b6453770 
   cmake/FindUTEMPTER.cmake PRE-CREATION 
   src/CMakeLists.txt caf2f0ba87ad4173af9860ae369b43d50ffd219f 
   src/ConfigureChecks.cmake f52be3f5e031c73dd7d26296622c14c9d69db42c 
 
 Diff: https://git.reviewboard.kde.org/r/122313/diff/
 
 
 Testing
 ---
 
 built, cmake configuration looks ok.
 
 
 Thanks,
 
 Hrvoje Senjan
 


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


Re: Review Request 122313: Expose to world whether KPty has been built with utempter library

2015-03-19 Thread Hrvoje Senjan

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

(Updated March 19, 2015, 11:18 a.m.)


Status
--

This change has been marked as submitted.


Review request for KDE Frameworks and David Faure.


Changes
---

Submitted with commit acf078d84d8f1c4d5fcf28cf9a9f570760ba1501 by Hrvoje Senjan 
to branch master.


Repository: kpty


Description
---

Equivalent to https://svn.reviewboard.kde.org/r/2468/
It was lost in KF5 porting, and it was directly used to determine whether 
kwrited should be built as module, or executable (in this case, it would be a 
SUID binary, which Qt5 no longer -by default- allows)


Diffs
-

  CMakeLists.txt 7fe77da7b0bc97c6f64db4fcc63ef7831fa065b1 
  KF5PtyConfig.cmake.in 04bde7bffd209b57e755a66278025ee8b6453770 
  cmake/FindUTEMPTER.cmake PRE-CREATION 
  src/CMakeLists.txt caf2f0ba87ad4173af9860ae369b43d50ffd219f 
  src/ConfigureChecks.cmake f52be3f5e031c73dd7d26296622c14c9d69db42c 

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


Testing
---

built, cmake configuration looks ok.


Thanks,

Hrvoje Senjan

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


Re: Review Request 122313: Expose to world whether KPty has been built with utempter library

2015-03-18 Thread Hrvoje Senjan

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


ping

- Hrvoje Senjan


On Jan. 29, 2015, 7:54 p.m., Hrvoje Senjan wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://git.reviewboard.kde.org/r/122313/
 ---
 
 (Updated Jan. 29, 2015, 7:54 p.m.)
 
 
 Review request for KDE Frameworks and David Faure.
 
 
 Repository: kpty
 
 
 Description
 ---
 
 Equivalent to https://svn.reviewboard.kde.org/r/2468/
 It was lost in KF5 porting, and it was directly used to determine whether 
 kwrited should be built as module, or executable (in this case, it would be a 
 SUID binary, which Qt5 no longer -by default- allows)
 
 
 Diffs
 -
 
   CMakeLists.txt 7fe77da7b0bc97c6f64db4fcc63ef7831fa065b1 
   KF5PtyConfig.cmake.in 04bde7bffd209b57e755a66278025ee8b6453770 
   cmake/FindUTEMPTER.cmake PRE-CREATION 
   src/CMakeLists.txt caf2f0ba87ad4173af9860ae369b43d50ffd219f 
   src/ConfigureChecks.cmake f52be3f5e031c73dd7d26296622c14c9d69db42c 
 
 Diff: https://git.reviewboard.kde.org/r/122313/diff/
 
 
 Testing
 ---
 
 built, cmake configuration looks ok.
 
 
 Thanks,
 
 Hrvoje Senjan
 


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


Review Request 122313: Expose to world whether KPty has been built with utempter library

2015-01-29 Thread Hrvoje Senjan

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

Review request for KDE Frameworks and David Faure.


Repository: kpty


Description
---

Equivalent to https://svn.reviewboard.kde.org/r/2468/
It was lost in KF5 porting, and it was directly used to determine whether 
kwrited should be built as module, or executable (in this case, it would be a 
SUID binary, which Qt5 no longer -by default- allows)


Diffs
-

  CMakeLists.txt 7fe77da7b0bc97c6f64db4fcc63ef7831fa065b1 
  KF5PtyConfig.cmake.in 04bde7bffd209b57e755a66278025ee8b6453770 
  cmake/FindUTEMPTER.cmake PRE-CREATION 
  src/CMakeLists.txt caf2f0ba87ad4173af9860ae369b43d50ffd219f 
  src/ConfigureChecks.cmake f52be3f5e031c73dd7d26296622c14c9d69db42c 

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


Testing
---

built, cmake configuration looks ok.


Thanks,

Hrvoje Senjan

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