-
Notifications
You must be signed in to change notification settings - Fork 20
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
Invalid machine names for desktop motherboards (?) #144
Comments
That's weird... That kernel also looks incorrect I think.
No worries, glad you're back. |
Oh yeah. I didn't notice the weird kernel at all. |
I'll try and see if there were any regressions in recent commits. I'd love to cherrypick my through the commits but I don't have a Windows machine to test with, we also don't know if this can be reproduced yet. |
Hmm I'm not sure if this is a bug specifically on windows. This happens on linux as well. Also this is not a oem manufactured desktop maybe that's why the relevant parts are placeholder text ? |
I went as far back as 5142f0a (a year ago) and nothing from the commit log looks suspicious, no changes were even introduced in the relevant functions. |
I suppose some machines just simply don't report their information correctly. There isn't, AFAIK, a good and sensible way to compose the "name" of machine. |
I don't know, the only way to tell is to use the Windows registry and hunt the components down and see whether the manufacturer bothered to name the product. |
I cat-ed the files in /sys/class/dmi/id I think the interesting ones are
Other ones which are probably causing the issue
|
What do you think we assume about non-OEM electronics that we can translate into a function that handles all these weird edge cases? Is there some sort of documentation we can follow to implement a one-size-fits-all function? I see here that the contents of |
Hmmm. I'm not really sure If these can be handled without having a at least a few examples of what /sys/class/dmi/id contains for a few people and then we'd also have to look into how to get those from windows as well. Can't really be sure without checking what other vendors fill in their product names as a place holder. |
@uttarayan21 this function from pfetch probably helps: https://github.com/dylanaraps/pfetch/blob/master/pfetch#L453 It basically uses a blacklist of known generic values and filters them out of the final string. I know of at least two computers that I have access to which also have a generic output, that might be enough to confirm which values need to be filtered. |
Oh that's very useful. |
Sure, that'd be great! |
Hello.
I was using windows 11 / Linux.
I installed
macchina
from master usingcargo install --git https://github.com/macchina-cli/macchina
On both machines it shows up as the following.
Edit: Added linux screenshot
Also sorry for the absence I have been really busy with work for a while. Hopefully I can find some time to work on some of the issues now.
Edit: It should be
Asus Z790-P
or similar but instead isASUS System Product Name
The text was updated successfully, but these errors were encountered: