search

nima / python-dmidecode

Python DMI-Decode
39 stars 27 forks source link

Use DWORD instead of WORD when calling dmi_memory_device_extended_size. #34

Closed mcepl closed 2 years ago

mcepl commented 2 years ago

I am a Pythonista, so I am not 100% certain why it works, this PR is more like making a record so it is not forgotten.

Fixes #33

copyninja commented 1 year ago

Why is this PR closed?. I still see issue when memory bank of 64GB python dmidecode prints 0TB where as actual dmidecode works

mcepl commented 1 year ago

I have given up on this dead repo and I didn’t want to keep around my fork of it which will never be used anyway. All patches used by openSUSE, can be downloaded from https://build.opensuse.org/package/show/openSUSE:Factory/python-dmidecode.

Hmm, this is not right, a SUSEman should replace upstream when it is dead. Let’s say, that my upstream will be from now on at https://git.sr.ht/~mcepl/dmidecode.

copyninja commented 1 year ago

Sorry my bad I was testing with one version old latest release 3.12.3 fixes it. Same change you did has been done in this release.