Use normalized version as keys for matching versions in maps

Author: cmaglie

.licenses/go/go.bug.st/relaxed-semver.dep.yml

@@ -1,6 +1,6 @@
 ---
 name: go.bug.st/relaxed-semver
-version: v0.9.0
+version: v0.9.1-0.20230314172648-aea0a9bb276e
 type: go
 summary: 
 homepage: https://pkg.go.dev/go.bug.st/relaxed-semver

arduino/libraries/librariesindex/index.go

@@ -35,7 +35,7 @@ var EmptyIndex = &Index{Libraries: map[string]*Library{}}
 // Library is a library available for download
 type Library struct {
 	Name     string
-	Releases map[string]*Release
+	Releases map[semver.NormalizedString]*Release
 	Latest   *Release `json:"-"`
 	Index    *Index   `json:"-"`
 }
@@ -117,7 +117,7 @@ func (idx *Index) FindRelease(ref *Reference) *Release {
 		if ref.Version == nil {
 			return library.Latest
 		}
-		return library.Releases[ref.Version.String()]
+		return library.Releases[ref.Version.NormalizedString()]
 	}
 	return nil
 }
@@ -173,13 +173,10 @@ func (idx *Index) ResolveDependencies(lib *Release) []*Release {
 
 // Versions returns an array of all versions available of the library
 func (library *Library) Versions() []*semver.Version {
-	res := []*semver.Version{}
-	for version := range library.Releases {
-		v, err := semver.Parse(version)
-		if err == nil {
-			res = append(res, v)
-		}
+	res := semver.List{}
+	for _, release := range library.Releases {
+		res = append(res, release.Version)
 	}
-	sort.Sort(semver.List(res))
+	sort.Sort(res)
 	return res
 }

arduino/libraries/librariesindex/json.go

@@ -90,7 +90,7 @@ func (indexLib *indexRelease) extractLibraryIn(index *Index) {
 	if !exist {
 		library = &Library{
 			Name:     indexLib.Name,
-			Releases: map[string]*Release{},
+			Releases: map[semver.NormalizedString]*Release{},
 		}
 		index.Libraries[indexLib.Name] = library
 	}
@@ -120,7 +120,7 @@ func (indexLib *indexRelease) extractReleaseIn(library *Library) {
 		License:          indexLib.License,
 		ProvidesIncludes: indexLib.ProvidesIncludes,
 	}
-	library.Releases[indexLib.Version.String()] = release
+	library.Releases[indexLib.Version.NormalizedString()] = release
 	if library.Latest == nil || library.Latest.Version.LessThan(release.Version) {
 		library.Latest = release
 	}

commands/lib/search.go

@@ -78,8 +78,8 @@ func indexLibraryToRPCSearchLibrary(lib *librariesindex.Library, omitReleasesDet
 	var releases map[string]*rpc.LibraryRelease
 	if !omitReleasesDetails {
 		releases = map[string]*rpc.LibraryRelease{}
-		for str, rel := range lib.Releases {
-			releases[str] = getLibraryParameters(rel)
+		for _, rel := range lib.Releases {
+			releases[rel.Version.String()] = getLibraryParameters(rel)
 		}
 	}
 

go.mod

@@ -35,7 +35,7 @@ require (
 	github.com/stretchr/testify v1.8.0
 	go.bug.st/cleanup v1.0.0
 	go.bug.st/downloader/v2 v2.1.1
-	go.bug.st/relaxed-semver v0.9.0
+	go.bug.st/relaxed-semver v0.9.1-0.20230314172648-aea0a9bb276e
 	go.bug.st/serial v1.3.2
 	golang.org/x/crypto v0.7.0
 	golang.org/x/text v0.8.0

go.sum

@@ -357,8 +357,8 @@ go.bug.st/cleanup v1.0.0 h1:XVj1HZxkBXeq3gMT7ijWUpHyIC1j8XAoNSyQ06CskgA=
 go.bug.st/cleanup v1.0.0/go.mod h1:EqVmTg2IBk4znLbPD28xne3abjsJftMdqqJEjhn70bk=
 go.bug.st/downloader/v2 v2.1.1 h1:nyqbUizo3E2IxCCm4YFac4FtSqqFpqWP+Aae5GCMuw4=
 go.bug.st/downloader/v2 v2.1.1/go.mod h1:VZW2V1iGKV8rJL2ZEGIDzzBeKowYv34AedJz13RzVII=
-go.bug.st/relaxed-semver v0.9.0 h1:qt0T8W70VCurvsbxRK25fQwiTOFjkzwC/fDOpyPnchQ=
-go.bug.st/relaxed-semver v0.9.0/go.mod h1:ug0/W/RPYUjliE70Ghxg77RDHmPxqpo7SHV16ijss7Q=
+go.bug.st/relaxed-semver v0.9.1-0.20230314172648-aea0a9bb276e h1:yuBa8KnRlbdxgjj9f+gLPjniQm7VGxPL34Z/cR8YKrY=
+go.bug.st/relaxed-semver v0.9.1-0.20230314172648-aea0a9bb276e/go.mod h1:lPVGdtzbQ9/2fv6iXqIXWHOj6cMTUJ/l/Lu1w+sgdio=
 go.bug.st/serial v1.3.2 h1:6BFZZd/wngoL5PPYYTrFUounF54SIkykHpT98eq6zvk=
 go.bug.st/serial v1.3.2/go.mod h1:jDkjqASf/qSjmaOxHSHljwUQ6eHo/ZX/bxJLQqSlvZg=
 go.bug.st/testifyjson v1.1.1 h1:nHotIMK151LF3vYsU/b2RaoVaWCgrf2kvQeGNoZkGaA=