-
Notifications
You must be signed in to change notification settings - Fork 19
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
loginutils/getty issues #2
Labels
enhancement
New feature or request
Comments
True. Please ignore my previous simple implementation. We need a decent |
Yeah, I'm just writing down some notes. The current version is a good enough jumping off point though. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Firstly,
getty
shouldn't just panic on failure (it should instead returnResult
s like the other utilities). Because of howgetty
affects the state of the system, I think theexecve()
call may be alright though.getty
should not be hard-coded to call/bin/ion
. It should instead take an argument specifying what TTY to take control of and what program to start on said TTY (the current values can serve as defaults, however). We may want to implementgettytab
or something similar as well.The text was updated successfully, but these errors were encountered: