-
Notifications
You must be signed in to change notification settings - Fork 188
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
Add an option to change the default shell. #215
base: master
Are you sure you want to change the base?
Conversation
I do not object to this. Configuration in this particular area might be useful. The previous PR posited "The default shell should be bash" which was wrong. This one posits "The default shell should be configurable on image creation" which seems reasonable. I'm pretty sure the commit message is incorrect. |
fc2b155
to
2c53fca
Compare
I've made the commit message clearer. |
This commit add a -S option that changes the default root shell of the iso. It also mentions the live.shell kernel option that changes the shell of the unprivileged user (anon).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This would be a neat feature to have
The option doesn't currently set the shell for the anon
user, should this also be implemented? I think it might be doable by modifying dracut/vmklive/adduser.sh
when creating the image.
-S <shell> Default shell of the root user (must be absolute path). | ||
Set the live.shell kernel parameter to change the default shell of anon. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since this PR was made, #267 was merged and added a -S argument.
This adds an option to set the default shell of the
root
user and documents thelive.shell
kernel option.I know that a similar change has already been closed (#99). My change makes this more configurable and optional. And it documents the
live.shell
option that isn't documented anywhere else and is pretty useful.I'll understand if you close this, but I consider this helpful for people who are customizing their iso.