<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 31, 2017 at 11:56 AM, Miika Turkia <span dir="ltr"><<a href="mailto:miika.turkia@gmail.com" target="_blank">miika.turkia@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Tue, Oct 31, 2017 at 12:31 PM, Martin Měřinský <<a href="mailto:mermar@centrum.cz">mermar@centrum.cz</a>> wrote:<br>
> Hello.<br>
> These days more and more distributions are using Wayland. Last Ubuntu<br>
> e.g. AFAIK Wayland design feature is that UI applications are not<br>
> allowed to run as root.<br>
><br>
> From Subsurface manual: "If you are not root, you may not be a member<br>
> of that group and won’t be able to use the USB port." Well, if you are<br>
> root, you cannot run Subsurface under Wayland at all(?) Fortunately<br>
> it's not necessary, but output doesn't look good.<br>
<br>
</span>We do not recommend running Subsurface as root and try to get users to<br>
configure their system so that root is not needed (i.e. adding<br>
themselves to the dialout group). That is the whole purpose of the<br>
specific chapter in user manual. But this sentence should be reworded<br>
to make it clear that user should set the groups properly and not run<br>
Subsurface as root.<br>
<span class=""><br>
> I would humbly suggest to detect Wayland and root privileges, show<br>
> better command line message and quit.<br>
> Or fix it somehow like GParted 0.30.0 did?<br>
> Maybe we should mention it in the manual.<br>
<br>
</span>There should not be any reason to run Subsurface as root, no matter<br>
whether Wayland or not. So any attempt to run with root privileges<br>
should result in an error message. But in Wayland's case it might be<br>
better not to mention the allow_run_as_root option at all (or tell the<br>
xhost cmdline).<br>
<span class=""><br>
> Also --allow_run_as_root option is not mentioned in subsurface --help<br>
> output nor in the manual.<br>
<br></span></blockquote><div><br></div><div>perhaps we should remove that, period.</div><div> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
</span>I am not sure if this should be documented or not. Running software as<br>
root is generally a bad idea, so documenting how to do that sounds a<br>
bit contradictory to me.<br>
<span class="HOEnZb"><font color="#888888"><br>
miika<br>
</font></span><div class="HOEnZb"><div class="h5">______________________________<wbr>_________________<br>
subsurface mailing list<br>
<a href="mailto:subsurface@subsurface-divelog.org">subsurface@subsurface-divelog.<wbr>org</a><br>
<a href="http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface" rel="noreferrer" target="_blank">http://lists.subsurface-<wbr>divelog.org/cgi-bin/mailman/<wbr>listinfo/subsurface</a><br>
</div></div></blockquote></div><br></div></div>