Page MenuHomeSolus
Differential D8903

Update cpu-x to 4.0.0
AbandonedPublic
Actions

Authored by algent on Sun, May 17, 8:20 PM.

Details

Reviewers
None
Group Reviewers
Triage Team
Summary

Changelog:

  • Added
    • Add a daemon to handle privileged access
    • Add basic completions for Bash/Fish/Zsh
    • Add settings window in GUI
    • Add "Driver" label in Graphics tab
    • Add support for zenpower module
  • Changed
    • Patch dmidecode to version 3.2.5b3c8e9
    • Uniform all units (byte and octet)
    • Rewrite all CLI options
    • Write output to /tmp/cpu-x.log and /tmp/cpu-x-daemon.log files when --issue-fmt is used
    • Update databases
    • Replace nvidia-settings command calls by nvidia-smi
    • Support for L1 Instruction Cache information
  • Removed
    • Remove all privileged access in cpu-x binary (moved to cpu-x-daemon)
    • Remove "CPU-X (Root)" desktop launcher
  • Fixed
    • Fix some awk regex
    • Fix load_module() function
    • Fix options parsing
    • Various C fixes (unsafe functions and warnings)
    • Fix with VFIO GPU driver
    • Fix GUI switches appearance in Bench tab
    • Various fixes in NCurses TUI

Signed-off-by: Algent Albrahimi <algent@protonmail.com>

Test Plan
  • Checked the info about my CPU.
  • Ran built-in benchmarks.

Diff Detail

Repository
R4515 cpu-x
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage

Event Timeline

algent created this revision.Sun, May 17, 8:20 PM
algent requested review of this revision.Sun, May 17, 8:20 PM
algent edited the summary of this revision. (Show Details)Sun, May 17, 8:21 PM
aleksvor added a subscriber: aleksvor.Mon, May 18, 9:57 AM

A few remarks about your changelog (as a person who packaged it and was going to update it):

Move translations from Transifex to Weblate

Not relevant to Solus users - that's a change only for translators who work on CPU-X.

Remove libcurl dependency
Remove libjson-c dependency

Not relevant because we never used them. These libraries were needed only for auto-updating CPU-X, so I never added them to builddeps to disable the auto-updater.

JoshStrobl added a subscriber: JoshStrobl.Mon, May 18, 10:14 AM

Maybe @aleksvor should be the maintainer of this instead of people just looking at stuff like repology? Kinda getting tired of drive-by updates.

algent edited the summary of this revision. (Show Details)Mon, May 18, 10:15 AM
algent added a comment.Mon, May 18, 10:18 AM

I use this app and there wasn't any MAINTAINERS.md file.
No problem abandoning this.

algent abandoned this revision.Mon, May 18, 10:43 AM

Leaving this to @aleksvor .

aleksvor added a comment.Mon, May 18, 10:55 AM

there wasn't any MAINTAINERS.md file.

Sorry about confusion, it's just because I made the last patch before maintainers file was a thing (CPU-X didn't have any updates in more than a year, and there was no need to touch this package all this time). I was going to claim maintainership with 4.0.0 update.
OK, I'll update this and add maintainers file later today. Actually, your patch already did everything right (CPU-X is light on build options and dependencies, so bumping version / release numbers and updating source is enough for it), only changelog was too verbose :).

Revision Contents
Changeset List

PathPackages
Mabi_used_libs (1 line)
Mpackage.yml (7 lines)
Mpspec_x86_64.xml (42 lines)

Diff 21440

View Options

abi_used_libs

Loading...
View Options

package.yml

Loading...
View Options

pspec_x86_64.xml

Loading...
Copyright © 2015-2019 Solus Project. The Solus logo is Copyright © 2016-2019 Solus Project. All Rights Reserved.