On Wed, Nov 1, 2017 at 4:27 PM, Martin Měřinský <[email protected]> wrote:
> On Tue, 2017-10-31 at 18:21 +0100, Robert C. Helling wrote: > > Hi, > > > > > Am 31.10.2017 um 12:51 schrieb Tomaz Canabrava <[email protected]> > > > : > > > > > > > > > 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. > > A lot of users don't know what they are doing, they are basically following internet recipes and running sudo $rand its important to prevent users shooting at their own foot. Another option would be enabling this feature compile-time. I do not > prefer this solution. > > M. > _______________________________________________ > subsurface mailing list > [email protected] > http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface >
_______________________________________________ subsurface mailing list [email protected] http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface
