8000 users: support on OpenBSD · Issue #5665 · uutils/coreutils · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

users: support on OpenBSD #5665

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.

Sign up for GitHub

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

Jump to bottom
Closed
lcheylus opened this issue Dec 17, 2023 · 2 comments · Fixed by #6406
Closed

users: support on OpenBSD #5665

lcheylus opened this issue Dec 17, 2023 · 2 comments · Fixed by #6406
Labels
U - users

Comments

@lcheylus
Copy link
Contributor

Following PR#5620

users tool is not supported on OpenBSD, due to the lack of support for utmpx (see #5596).

=> implement users on OpenBSD with an alternate solution without using utmpx.
@jadijadi
Copy link
Contributor

I would be glad to work on this. I'm an occasional OpenBSD user and tried compiling coreutils there; which worked smoothly, But obviously no users (and pingy, who, ...).

Since this is a larger contribution than what I've done before on this project, I've decided to ask first & propose my solution.

My idea is adding a utmp module alongside utmpx and use it when retrieving data from related files (say /var/run/utmp for users).

Would be glad to have your comments & suggestions.

@lcheylus
Copy link
Contributor Author

Great news if I'm not the only dev to work on OpenBSD support for coreutils :)

I have also a partial implementation of users with a new utmp module to grep infos in /var/run/utmp. I think this is the right solution. I would be interested to review your code if you propose a PR.

(WIP for me to add CI support for OpenBSD)

jadijadi added a commit to jadijadi/coreutils that referenced this issue May 16, 2024
@jadijadi
users: added to the openbsd
4ed9afd 
OpenBSD uses the original utmp file format so the general `users` does not
work there. By using the `utmp-classic` lib, we cna process the current users
on the system from the /var/run/utmp and show the list.

fixes uutils#5665
@jadijadi jadijadi mentioned this issue May 16, 2024
Merged
jadijadi added a commit to jadijadi/coreutils that referenced this issue May 19, 2024
@jadijadi
users: added to the openbsd
e13c08a 
OpenBSD uses the original utmp file format so the general `users` does not
work there. By using the `utmp-classic` lib, we cna process the current users
on the system from the /var/run/utmp and show the list.

fixes uutils#5665
jadijadi added a commit to jadijadi/coreutils that referenced this issue May 21, 2024
@jadijadi
users: support OpenBS using utmp
89e4406 
OpenBSD uses the original utmp file format so the general `users` does not
work there. By using the `utmp-classic` lib, we cna process the current users
on the system from the /var/run/utmp and show the list.

fixes uutils#5665
sylvestre pushed a commit to jadijadi/coreutils that referenced this issue Jun 25, 2024
@jadijadi @sylvestre
users: support OpenBS using utmp
c307fd1 
OpenBSD uses the original utmp file format so the general `users` does not
work there. By using the `utmp-classic` lib, we cna process the current users
on the system from the /var/run/utmp and show the list.

fixes uutils#5665
jadijadi added a commit to jadijadi/coreutils that referenced this issue Jun 29, 2024
@jadijadi
users: support OpenBS using utmp
e6b3755 
OpenBSD uses the original utmp file format so the general `users` does not
work there. By using the `utmp-classic` lib, we cna process the current users
on the system from the /var/run/utmp and show the list.

fixes uutils#5665
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
U - users
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

users: support OpenBSD using utmp jadijadi/coreutils
3 participants
@cakebaker @jadijadi @lcheylus
0