Avoid using the newgrp command-line tool in the doc#25301
Conversation
✅ Deploy Preview for docsdocker ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
|
||
| ```console | ||
| $ newgrp docker | ||
| $ sudo -u $USER bash |
There was a problem hiding this comment.
Thanks for fixing the newgrp dependency problem. Is sudo -u $USER bash known to refresh group membership like newgrp docker does, uniformly across distributions? Maybe it's worth clarifying this for users following the post-installation steps.
There was a problem hiding this comment.
uniformly across distributions?
Yes.
Maybe it's worth clarifying this for users following the post-installation steps.
What are you talking about? Isn't this exactly what current PRs are doing?
There was a problem hiding this comment.
Thanks for the clarification. My concern was mainly whether readers would understand that sudo -u $USER bash is intended to refresh the session with the updated group membership. If that's already clear from the surrounding documentation, then no further clarification is needed from my side.
Description
util-linux-extrato usenewgrp dockerroadmap#910 .newgrpcommand-line tool in the doc. I don't think requiring users to install an additional apt package is a good option.Related issues or tickets
util-linux-extrato usenewgrp dockerroadmap#910Reviews