Root and Wayland

[Martin Měřinský]

On Tue, 2017-10-31 at 18:21 +0100, Robert C. Helling wrote:
> Hi,
> 
> > Am 31.10.2017 um 12:51 schrieb Tomaz Canabrava <tcanabrava at kde.org>
> > :
> > 
> > 
> > perhaps we should remove that, period.
> 
> A while ago, we had problems with people who at some point ran
> subsurface as root and the touched all kinds of files which meant
> later it did no longer run under the normal user. So I added the test
> and, since I didn’t want to for it users to shoot in their foot if
> they insisted on that introduced this command line option. So from
> that perspective it could simply go. 
> 
> The only reason I see for running with root privileges is checking if
> some problem is really because of lacking permissions. But still
> that’s not a good excuse. 

Thank you for your reply.

I don't see what is undocumented feature good for. From my point of
view, it would be better to remove such feature, disable to run
Subsurface as root and fix error messages.

On the other hand, I personally don't like software preventing user
from doing something he wants, if there is no technical reason to.
Running Subsurface as root under Wayland can be fixed (using polkig?).
And the necessity to use --allow_run_as_root option is very good and
very discouraging ;-) Document such feature with warning should be good
enough.

Another option would be enabling this feature compile-time. I do not
prefer this solution.

M.