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

New package: nvtop-1.2.2 #33460

Closed
wants to merge 1 commit into from
Closed

Conversation

Daxot
Copy link
Contributor

@Daxot Daxot commented Oct 10, 2021

General

Have the results of the proposed changes been tested?

  • I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
  • I generally don't use the affected packages but briefly tested this PR

@paper42 paper42 added the new-package This PR adds a new package label Oct 10, 2021
archs="x86_64"
build_style=cmake
makedepends="ncurses-devel"
depends="nvidia ncurses"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You don't need to list ncurses in depends, it will be detected.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this work with nvidia390 as well? You could drop the nvidia requirement as well for those on older systems. If you do that, set archs="x86_64 i686".

Copy link
Contributor Author

@Daxot Daxot Nov 7, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, and thank you for review.
About nvidia390: nvtop uses NVML Library which comes with nvidia drivers and as developer stated in github readme:
"The NVML library does not support some of the queries for GPUs coming before the Kepler microarchitecture. Anything starting at GeForce 600, GeForce 800M and successor should work fine".
So, i don't think that this package will be working correctly on nvidia390 and their GPU series.

@abenson abenson closed this in 4069446 Nov 7, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 5, 2022
@Daxot Daxot deleted the new-package-nvtop branch May 2, 2022 10:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
new-package This PR adds a new package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants