Johni0702 / mumble-web

An HTML5 Mumble client
674 stars 151 forks source link

Add explicit mention of build failures when running as the root users #113

Closed robozman closed 4 years ago

robozman commented 4 years ago

The readme currently states:

Note that npm should not be ran as root, use an unprivileged user account instead.

I initially interpreted this as a warning for security reasons and stupidly ignored it as I was running in a Docker container. I think it would be helpful to note here that the build literally doesn't work if you are the root user.

Is this a change you would be interested in adding to the readme? I can send a PR if so.