Skip to content
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

User utility along with non-privileged user installation script #134

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

jeff999955
Copy link

I packed the dependency within the repo and provided the building script for non-root users.

Moreover, I added the user utility function so that users could select whom to be monitored in the terminal.

The user utility is built on top of the original code with slight modification on function interface and some additional stuff.

@Syllo
Copy link
Owner

Syllo commented Jun 4, 2022

Hey @jeff999955,

So sorry I saw your pull request some time ago but forgot to follow up.

I won't merge the ncurses source because I don't think that it belongs in this repository, I'll probably add a link to the release page instead.
I will gladly cherry pick the bits for sorting/filtering by user name and the install instruction for a user without system install permissions.

Thank you for contributing to nvtop.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants