Version.LessThan() function not working properly

[josh-hogle]

Description

When using v0.19.0 of the Go library with v0.19.0 of the C library, I get the following warning message:

Note: the loaded ObjectBox C library should be updated.
      Found ObjectBox version 0.19.0, but the minimum recommended version is 0.18.1.

Basic info

Please complete the following information:

• ObjectBox version (are you using the latest version?): v0.19.0 (Go and C libraries)
• Reproducibility: always
• Device: desktop
• OS: Debian

How to reproduce

Simply build a small demo program similar to the getting started example and compile it and you should get the warning.

Expected behavior

No warning(s) should be emitted as the "C" library is greater than/equal to the Go library version.

Code

Check the LessThan() function in version.go on line 35. It appears to me that this may be causing the issue. You may also want to just use the golang.org/x/mod/semver package to simplify your version comparison and management.

[levin-go]

I have also discovered the same issue. When I was running the latest version on armv7, I encountered a panic

the code in version.go

func (v Version) LessThan(other Version) bool {
  return v.Major < other.Major || v.Minor < other.Minor || v.Patch < other.Patch
}

should be change to ?

func (v Version) LessThan(other Version) bool {
  if v.Major > other.Major {
    return false
  }
  if v.Minor > other.Minor {
    return false
  }
  if v.Patch > other.Patch {
    return false
  }
  if (v.Major == other.Major && v.Minor == other.Minor && v.Patch == other.Patch){
    return false
  }
  return true
}