Re: Review Request 117125: start_kdeinit: Use capabilities instead of SUID

2014-05-29 Thread Alex Merry
On April 11, 2014, 4:46 p.m., Commit Hook wrote: This review has been submitted with commit e898d13b430692e775060d49342181192e122fdf by Hrvoje Senjan to branch master. Hrvoje Senjan wrote: i've reverted the commit now. capabilities break LD_LIBRARY_PATH, so this is a no-go.

Re: Review Request 117125: start_kdeinit: Use capabilities instead of SUID

2014-05-29 Thread Alex Merry
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117125/#review58706 --- What's the plan with this? Does Andreas' fix for the setuid

Re: Review Request 117125: start_kdeinit: Use capabilities instead of SUID

2014-05-29 Thread Hrvoje Senjan
On May 29, 2014, 2:11 p.m., Alex Merry wrote: What's the plan with this? Does Andreas' fix for the setuid case also fix the capabilities case? Does Andreas' fix for the setuid case also fix the capabilities case? yep. i was able to successfully start and use plasma next (with KF5 in /usr,

Re: Review Request 117125: start_kdeinit: Use capabilities instead of SUID

2014-05-29 Thread Hrvoje Senjan
On April 11, 2014, 6:46 p.m., Commit Hook wrote: This review has been submitted with commit e898d13b430692e775060d49342181192e122fdf by Hrvoje Senjan to branch master. Hrvoje Senjan wrote: i've reverted the commit now. capabilities break LD_LIBRARY_PATH, so this is a no-go.

Re: Review Request 117125: start_kdeinit: Use capabilities instead of SUID

2014-05-29 Thread Alex Merry
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117125/#review58714 --- Ship it! Given that the original issue seems to be fixed, I

Re: Review Request 117125: start_kdeinit: Use capabilities instead of SUID

2014-05-29 Thread Hrvoje Senjan
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117125/ --- (Updated May 29, 2014, 3:14 p.m.) Review request for KDE Frameworks,

Re: Review Request 117125: start_kdeinit: Use capabilities instead of SUID

2014-05-29 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117125/#review58742 --- This review has been submitted with commit

Re: Review Request 117125: start_kdeinit: Use capabilities instead of SUID

2014-05-29 Thread Hrvoje Senjan
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117125/ --- (Updated May 29, 2014, 3:55 p.m.) Status -- This change has been

Re: Review Request 117125: start_kdeinit: Use capabilities instead of SUID

2014-05-15 Thread Hrvoje Senjan
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117125/ --- (Updated May 15, 2014, 11:12 p.m.) Review request for KDE Frameworks,

Re: Review Request 117125: start_kdeinit: Use capabilities instead of SUID

2014-04-14 Thread Hrvoje Senjan
On April 11, 2014, 4:46 p.m., Commit Hook wrote: This review has been submitted with commit e898d13b430692e775060d49342181192e122fdf by Hrvoje Senjan to branch master. Hrvoje Senjan wrote: i've reverted the commit now. capabilities break LD_LIBRARY_PATH, so this is a no-go.

Re: Review Request 117125: start_kdeinit: Use capabilities instead of SUID

2014-04-11 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117125/#review55438 --- Ship it! Ship It! - David Faure On April 7, 2014, 7:05

Re: Review Request 117125: start_kdeinit: Use capabilities instead of SUID

2014-04-11 Thread Hrvoje Senjan
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117125/ --- (Updated April 11, 2014, 4:46 p.m.) Status -- This change has been

Re: Review Request 117125: start_kdeinit: Use capabilities instead of SUID

2014-04-11 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117125/#review55468 --- This review has been submitted with commit

Re: Review Request 117125: start_kdeinit: Use capabilities instead of SUID

2014-04-11 Thread Hrvoje Senjan
On April 11, 2014, 4:46 p.m., Commit Hook wrote: This review has been submitted with commit e898d13b430692e775060d49342181192e122fdf by Hrvoje Senjan to branch master. i've reverted the commit now. capabilities break LD_LIBRARY_PATH, so this is a no-go. apologies for potentially caused

Re: Review Request 117125: start_kdeinit: Use capabilities instead of SUID

2014-04-07 Thread Hrvoje Senjan
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117125/ --- (Updated April 7, 2014, 7:05 p.m.) Review request for KDE Frameworks and

Re: Review Request 117125: start_kdeinit: Use capabilities instead of SUID

2014-03-28 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117125/#review54461 --- src/start_kdeinit/start_kdeinit.c

Review Request 117125: start_kdeinit: Use capabilities instead of SUID

2014-03-27 Thread Hrvoje Senjan
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117125/ --- Review request for KDE Frameworks and David Faure. Bugs:

Re: Review Request 117125: start_kdeinit: Use capabilities instead of SUID

2014-03-27 Thread Alex Merry
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117125/#review54374 --- Can you have a look at the documentation about writing a