search

clbr / radeontop

GNU General Public License v3.0
789 stars 69 forks source link

New release? #94

Open FFY00 opened 4 years ago

FFY00 commented 4 years ago

Hey, could we get a new release to add support for Navi GPUs?

clbr commented 4 years ago

It's not a good point for a release. Please use the git version if you need navi for now.

bjornfor commented 2 years ago

How about creating a release now?

I realize this issue was created before the last release (1.3), but that release happened in 2020-12 and today is 2021-11 (almost a year ago). PCI IDs for the latest cards is nice to have, I'm currently getting Unknown Radeon card. <= R500 won't work, new cards might. (for 6700 XT card). This warning required some investigation, which led me here. If you create a release we (community) can update linux distro packages and users don't have to do ad-hoc installs from git.

clbr commented 2 years ago

Tag pushed. It is a bit silly for distros to want a named release, for apps that don't do anything for said release (radeontop releases are just tags for whatever git commit is latest and works fine).

bjornfor commented 2 years ago

Thanks!

danfe commented 2 years ago

Tag pushed. It is a bit silly for distros to want a named release, for apps that don't do anything for said release

It's not silly. Downstream package maintainers prefer stable tags by large margin (ideally, properly bootstrapped releases). It gives better perspective of the project's pace and progress, without creating arbitrary "snapshots" when they have to be e.g. date-based as there are no other strong anchors. FWIW, in v1.4 new translations (NLS catalogues) were added and the program had been moved to bin from sbin, both of which are user-visible changes (modulo obvious improvements in hardware support).

That said, thanks for tagging v1.4! :-)