gsmartcontrol icon indicating copy to clipboard operation
gsmartcontrol copied to clipboard

Pre-releases of smartctl no longer work due to "pre-" prefix in version info

Open chrfranke opened this issue 2 years ago • 4 comments

Version and Environment

  • GSmartControl version: 1.1.4
  • OS: Windows 10 22H2 (but affects all platforms)

Describe the Bug Pre-releases of smartctl no longer work due to "pre-" prefix in version info

To Reproduce Steps to reproduce the behavior:

  1. Install a CI release from https://builds.smartmontools.org/ or build it from source.
  2. Select "Look for smartctl in smartmontools installation directory first" in GSmartControl preferences.
  3. Press ^R
  4. "Unknown model" is shown for all drives.

Additional Context First smartctl output line of CI builds after smartmontools 7.3: smartctl pre-7.4 2023-06-13 r5481 [x86_64-w64-mingw32-w10-22H2] (CircleCI)

If the "pre-" is removed, GSmartControl works again. So current GSmartControl 1.1.4 will work with upcoming smartmontools 7.4 release.

chrfranke avatar Jun 20 '23 14:06 chrfranke

Hi Christian,

Thanks for letting me know. I will fix this in the main branch. I'm not sure it's critical enough to be backported to 1.1 branch though.

Thank you!

ashaduri avatar Jun 21 '23 11:06 ashaduri

You're welcome. Backporting is IMO not needed as there is possibly no benefit for GSmartControl's use cases with a 7.4 pre-release instead of a 7.3 release and with the upcoming 7.4 release it will work again.

chrfranke avatar Jun 21 '23 14:06 chrfranke

FYI: New official release 7.4 works with GSmartControl 1.1.4, so no need to backport anything.

OT: In next GSmartControl release, you might want to show the drive database version number instead of Yes/No. The related smartctl line is now for example:

Device is:        In smartctl database 7.3/5528
Device is:        Not in smartctl database 7.3/5528

chrfranke avatar Aug 01 '23 12:08 chrfranke

Christian, thanks for the suggestion, I should display the version there. Hopefully, I will have enough time to finally finish the JSON port as well.

ashaduri avatar Aug 01 '23 13:08 ashaduri

This has now been implemented in the main branch.

Thanks!

ashaduri avatar May 16 '24 12:05 ashaduri