-
Notifications
You must be signed in to change notification settings - Fork 37
feat: macos support #12
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
base: master
Are you sure you want to change the base?
Conversation
|
Thanks, I will test this out. I am liking this idea of having a separate file for each "group" of operating systems. i.e., darwin, linux, bsd, etc. |
Heya @AnarchistHoneybun. I'll double check this later this evening (BST). When you have a moment, could you post the output of Here's what mine outputs: Reference(s):
|
Hi @neilpanchal! Love your creative work (USGC) on "Twitter", which brought me to your repo. Yes, I figured this approach would reduce maintenance and allow for platform-specific tweaks and features where needed. Additionally, a main Bash script can identify the appropriate platform-specific file to run. This mini-modularity could prove useful. Hope you like it. |
sorry for getting back on this so late, I ran the command and then forgot to post the results for some reason. It's the exact same field and formatting as yours, so I don't think that's a problem. that particular section is supposed to show who logged in last to the system, and afaik it also handles remote logins (my above screenshot is from an ubuntu machine in one such session). You might want to check how your program is extracting the last_login_ip field if that's the functionality you intend too, and if not, it's probably better to drop/rename that field in the script to avoid confusion |


Quick and dirty macOS support.
Early. Likely buggy.