New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Don't suppress the shutdown menu on LTSP clients #535
Comments
@lukefromdc @alkisg |
@raveit65, LTSP (ltsp.org, Linux Terminal Server Project) is a collection of packages that helps sysadmins set up DHCP, TFTP, NBD, SSHFS etc, so that diskless clients can be booted from the network.
I'm testing with mate-panel's redmond layout, because the ubuntu-mate layout doesn't include the shutdown menu item anyway. |
I support reverting this; it won't affect anyone other than LTSP clients, and we don't need that in there anymore. |
I can confirm that mate-panel in master as of today suppresses the shutdown item in the main menu (shutdown applet from mate-applets unaffected) when restarted with |
This commit, 9 years ago, aimed at hiding the shutdown menu on LTSP clients, because they were remote sessions to the LTSP server, and we didn't want remote users to shut down the server by accident.
Nowadays it prevents LTSP fat clients from shutting down. Fat clients is the recommended LTSP setup, and they run the session locally on the client, so having a shutdown button there is required.
As for thin clients, preventing server shutdown is taken care of by policykit, there's no need to check for an environment variable in the MATE code (btw I'm an LTSP developer).
So, I'd like to ask you to please revert commit 8c25e1b.
The text was updated successfully, but these errors were encountered: