Correction for benchmarks not running on Arch

[westonmyers]

L.,

I went to run hardinfo on Arch and was unable to complete any benchmarks. High CPU usage was observed and the application would stay at the benchmarking dialog for as long as you'd let it.

Decided to check if others were having issues as well and stumbled into people in the Arch User Repository commenting along with others here in the upstream. (#640, etc)

straced it down to a module dependency issue; as seen below.

I just wanted to push in at least a quick modification for Arch peeps and open dialog here. AUR is running a patch of the below for now as the maintainer there was able to confirm the ability to run benchmarks when patched.

I'll see about testing other distros, etc. Ideally checks would be added for this module issue. There were also some aberrations seen while testing that need to be debugged as well. Starting with something is better than nothing though!

Thanks for your time,
Weston

[lpereira]

Thanks! I think that module dependencies might be broken ("-a" switch to autoload dependencies) if your patch is any indication. I'm far away from my computer but I'll take a look next week.

…

On Fri, Apr 29, 2022, at 12:40 PM, Weston Myers wrote: L., I went to run hardinfo on Arch and was unable to complete any benchmarks. High CPU usage was observed and the application would stay at the benchmarking dialog for as long as you'd let it. Decided to check if others were having issues as well and stumbled into people in the Arch User Repository <https://aur.archlinux.org/packages/hardinfo-git> commenting along with others here in the upstream. (#640 <#640>, etc) straced it down to a module dependency issue; as seen below. I just wanted to push in at least a quick modification for Arch peeps and open dialog here. AUR is running a patch of the below for now as the maintainer there was able to confirm the ability to run benchmarks when patched. I'll see about testing other distros, etc. Ideally checks would be added for this module issue. There were also some aberrations seen while testing that need to be debugged as well. Starting with something is better than nothing though! Thanks for your time, Weston You can view, comment on, or merge this pull request online at: #646 Commit Summary * ec85aad <ec85aad> Quick and dirty fix to re-enable benchmarking. Patch currently in Arch User Repository: https://aur.archlinux.org/packages/hardinfo-git File Changes (1 file <https://github.com/lpereira/hardinfo/pull/646/files>) * *M* modules/benchmark.c <https://github.com/lpereira/hardinfo/pull/646/files#diff-3454ef879ab0d49212d9af68b6ac521e12909ed3df0bcf3afdfd2937d60453df> (4) Patch Links: * https://github.com/lpereira/hardinfo/pull/646.patch * https://github.com/lpereira/hardinfo/pull/646.diff — Reply to this email directly, view it on GitHub <#646>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAADVGITOBEC2MIQINVE62TVHQUJ7ANCNFSM5UW5AMGA>. You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>