FabricMC / fabric-installer

An Installer for Fabric. For the vanilla launcher
Apache License 2.0
102 stars 65 forks source link

Suggestion: Add a warning on the installer for when the installer is run with sudo privileges on Linux #112

Open DNAmaster10 opened 1 year ago

DNAmaster10 commented 1 year ago

Sorry if this is the wrong place to put this!

I'd like to suggest the addition of a warning to the installer when someone runs it as sudo. Running it as sudo can create all sorts of problems down the line for users, and cause the launcher to not work at all. I made a small post about this before, which has recently become active after someone tried to do this (here is the Reddit post). For new Linux users, who may not understand the difference between running something as administrator on Windows vs running it as sudo on Linux, it can be quite the issue.

Thanks!

modmuss50 commented 1 year ago

Good idea 👍

It should also apply to when running the installer as admin on windows and sudo on MacOS.

Ayush-Thakur-geek commented 4 months ago

hey @modmuss50 could you pls verify the improvements in the pr.