Skip to content
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

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

Closed
chrfranke opened this issue Jun 20, 2023 · 5 comments
Assignees
Labels
bug Something isn't working enhancement New feature or request fixed The issue has been fixed

Comments

@chrfranke
Copy link

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 chrfranke added bug Something isn't working enhancement New feature or request labels Jun 20, 2023
@ashaduri
Copy link
Owner

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!

@chrfranke
Copy link
Author

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
Copy link
Author

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

@ashaduri
Copy link
Owner

ashaduri commented Aug 1, 2023

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 ashaduri added the fixed The issue has been fixed label May 16, 2024
@ashaduri
Copy link
Owner

This has now been implemented in the main branch.

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request fixed The issue has been fixed
Projects
None yet
Development

No branches or pull requests

2 participants