Compatibility is now shown when calling lib list with fqbn

Author: silvanocerza

arduino/libraries/libraries.go

@@ -74,6 +74,7 @@ type Library struct {
 	Properties             *properties.Map
 	Examples               paths.PathList
 	sourceHeaders          []string
+	Compatible             bool
 }
 
 func (library *Library) String() string {

cli/lib/list.go

@@ -108,10 +108,22 @@ func (ir installedResult) String() string {
 	})
 
 	t := table.New()
-	t.SetHeader("Name", "Installed", "Available", "Location", "Description")
+	header := []interface{}{}
+	if listFlags.fqbn == "" {
+		header = []interface{}{
+			"Name", "Installed", "Available", "Location", "Description",
+		}
+		t.SetColumnWidthMode(4, table.Average)
+	} else {
+		header = []interface{}{
+			"Name", "Installed", "Available", "Compatible", "Location", "Description",
+		}
+		t.SetColumnWidthMode(3, table.Average)
+		t.SetColumnWidthMode(5, table.Average)
+	}
 	t.SetColumnWidthMode(1, table.Average)
 	t.SetColumnWidthMode(2, table.Average)
-	t.SetColumnWidthMode(4, table.Average)
+	t.SetHeader(header...)
 
 	lastName := ""
 	for _, libMeta := range ir.installedLibs {
@@ -139,8 +151,18 @@ func (ir installedResult) String() string {
 			} else if len(sentence) > 40 {
 				sentence = sentence[:37] + "..."
 			}
-
-			t.AddRow(name, lib.Version, available, location, sentence)
+			if listFlags.fqbn == "" {
+				t.AddRow(name, lib.Version, available, location, sentence)
+			} else {
+				var compatible string
+				if lib.Compatible {
+					compatible = "✓"
+				} else {
+					compatible = "✕"
+				}
+
+				t.AddRow(name, lib.Version, available, compatible, location, sentence)
+			}
 		}
 	}
 

commands/lib/list.go

@@ -50,57 +50,69 @@ func LibraryList(ctx context.Context, req *rpc.LibraryListReq) (*rpc.LibraryList
 
 	instaledLib := []*rpc.InstalledLibrary{}
 	res := listLibraries(lm, req.GetUpdatable(), req.GetAll())
-	if len(res) > 0 {
-		if f := req.GetFqbn(); f != "" {
-			fqbn, err := cores.ParseFQBN(req.GetFqbn())
-			if err != nil {
-				return nil, fmt.Errorf("parsing fqbn: %s", err)
-			}
-			_, boardPlatform, _, _, refBoardPlatform, err := pm.ResolveFQBN(fqbn)
-			if err != nil {
-				return nil, fmt.Errorf("loading board data: %s", err)
-			}
+	if f := req.GetFqbn(); f != "" {
+		fqbn, err := cores.ParseFQBN(req.GetFqbn())
+		if err != nil {
+			return nil, fmt.Errorf("parsing fqbn: %s", err)
+		}
+		_, boardPlatform, _, _, refBoardPlatform, err := pm.ResolveFQBN(fqbn)
+		if err != nil {
+			return nil, fmt.Errorf("loading board data: %s", err)
+		}
 
-			filteredRes := map[string]*installedLib{}
-			for _, lib := range res {
-				if cp := lib.Library.ContainerPlatform; cp != nil {
-					if cp != boardPlatform && cp != refBoardPlatform {
-						// Filter all libraries from extraneous platforms
-						continue
-					}
+		filteredRes := map[string]*installedLib{}
+		for _, lib := range res {
+			if cp := lib.Library.ContainerPlatform; cp != nil {
+				if cp != boardPlatform && cp != refBoardPlatform {
+					// Filter all libraries from extraneous platforms
+					continue
 				}
-				if latest, has := filteredRes[lib.Library.Name]; has {
-					if latest.Library.LocationPriorityFor(boardPlatform, refBoardPlatform) >= lib.Library.LocationPriorityFor(boardPlatform, refBoardPlatform) {
-						continue
-					}
+			}
+			if latest, has := filteredRes[lib.Library.Name]; has {
+				if latest.Library.LocationPriorityFor(boardPlatform, refBoardPlatform) >= lib.Library.LocationPriorityFor(boardPlatform, refBoardPlatform) {
+					continue
 				}
-				filteredRes[lib.Library.Name] = lib
 			}
 
-			res = []*installedLib{}
-			for _, lib := range filteredRes {
-				res = append(res, lib)
+			// Check if library is compatible with board specified by FBQN
+			lib.Library.Compatible = false
+			for _, arch := range lib.Library.Architectures {
+				if arch == fqbn.PlatformArch || arch == "*" {
+					lib.Library.Compatible = true
+					break
+				}
 			}
+
+			filteredRes[lib.Library.Name] = lib
 		}
 
+		res = []*installedLib{}
+		for _, lib := range filteredRes {
+			res = append(res, lib)
+		}
+	} else {
+		// If no FQBN has been specified mark every library as compatible
 		for _, lib := range res {
-			if nameFilter != "" && strings.ToLower(lib.Library.Name) != nameFilter {
-				continue
-			}
-			libtmp, err := GetOutputLibrary(lib.Library)
-			if err != nil {
-				return nil, err
-			}
-			release := GetOutputRelease(lib.Available)
-			instaledLib = append(instaledLib, &rpc.InstalledLibrary{
-				Library: libtmp,
-				Release: release,
-			})
+			lib.Library.Compatible = true
 		}
+	}
 
-		return &rpc.LibraryListResp{InstalledLibrary: instaledLib}, nil
+	for _, lib := range res {
+		if nameFilter != "" && strings.ToLower(lib.Library.Name) != nameFilter {
+			continue
+		}
+		libtmp, err := GetOutputLibrary(lib.Library)
+		if err != nil {
+			return nil, err
+		}
+		release := GetOutputRelease(lib.Available)
+		instaledLib = append(instaledLib, &rpc.InstalledLibrary{
+			Library: libtmp,
+			Release: release,
+		})
 	}
-	return &rpc.LibraryListResp{}, nil
+
+	return &rpc.LibraryListResp{InstalledLibrary: instaledLib}, nil
 }
 
 // listLibraries returns the list of installed libraries. If updatable is true it
@@ -176,6 +188,7 @@ func GetOutputLibrary(lib *libraries.Library) (*rpc.Library, error) {
 		License:           lib.License,
 		Examples:          lib.Examples.AsStrings(),
 		ProvidesIncludes:  libHeaders,
+		Compatible:        lib.Compatible,
 	}, nil
 }
 

rpc/commands/lib.pb.go

@@ -271,6 +271,9 @@ message Library {
     // Value of the `includes` field in library.properties or, if missing, the list of
     // include files available on the library source root directory.
     repeated string provides_includes = 27;
+    // True in case this library is compatible with the specified board.
+    // Always true if no board is specified
+    bool compatible = 28;
 }
 
 enum LibraryLayout {
@@ -287,7 +290,7 @@ enum LibraryLocation {
     user = 1;
     // In the `libraries` subdirectory of a platform.
     platform_builtin = 2;
-    // When `LibraryLocation` is used in a context where a board is specified, 
+    // When `LibraryLocation` is used in a context where a board is specified,
     // this indicates the library is in the `libraries` subdirectory of a
     // platform referenced by the board's platform.
     referenced_platform_builtin = 3;

rpc/commands/lib.proto

@@ -73,6 +73,89 @@ def test_list(run_command):
     assert "Braccio.h" == data[0]["library"]["provides_includes"][0]
 
 
+def test_list_exit_code(run_command):
+    # Init the environment explicitly
+    assert run_command("core update-index")
+
+    assert run_command("core list")
+
+    # Verifies lib list doesn't fail when platform is not specified
+    result = run_command("lib list")
+    assert result.ok
+    assert result.stderr.strip() == ""
+
+    # Verify lib list command fails because specified platform is not installed
+    result = run_command("lib list -b arduino:samd:mkr1000")
+    assert result.failed
+    assert (
+        result.stderr.strip() == "Error listing Libraries: loading board data: platform arduino:samd is not installed"
+    )
+
+    assert run_command('lib install "AllThingsTalk LoRaWAN SDK"')
+
+    # Verifies lib list command keeps failing
+    result = run_command("lib list -b arduino:samd:mkr1000")
+    assert result.failed
+    assert (
+        result.stderr.strip() == "Error listing Libraries: loading board data: platform arduino:samd is not installed"
+    )
+
+    assert run_command("core install arduino:samd")
+
+    # Verifies lib list command now works since platform has been installed
+    result = run_command("lib list -b arduino:samd:mkr1000")
+    assert result.ok
+    assert result.stderr.strip() == ""
+
+
+def test_list_with_fqbn(run_command):
+    # Init the environment explicitly
+    assert run_command("core update-index")
+
+    # Install core
+    assert run_command("core install arduino:samd")
+
+    # Install some library
+    assert run_command("lib install ArduinoJson")
+    assert run_command("lib install wm8978-esp32")
+
+    # Look at the plain text output
+    result = run_command("lib list -b arduino:samd:mkr1000")
+    assert result.ok
+    assert "" == result.stderr
+    lines = result.stdout.strip().splitlines()
+    assert 3 == len(lines)
+
+    # Verifies library is compatible
+    toks = [t.strip() for t in lines[1].split(maxsplit=5)]
+    assert 6 == len(toks)
+    assert "ArduinoJson" == toks[0]
+    assert "✓" == toks[3]
+
+    # Verifies library is not compatible
+    toks = [t.strip() for t in lines[2].split(maxsplit=5)]
+    assert 6 == len(toks)
+    assert "wm8978-esp32" == toks[0]
+    assert "✕" == toks[3]
+
+    # Look at the JSON output
+    result = run_command("lib list -b arduino:samd:mkr1000 --format json")
+    assert result.ok
+    assert "" == result.stderr
+    data = json.loads(result.stdout)
+    assert 2 == len(data)
+
+    data = sorted(data, key=lambda lib: str.lower(lib["library"]["name"]))
+
+    # Verifies library is compatible
+    assert data[0]["library"]["name"] == "ArduinoJson"
+    assert data[0]["library"]["compatible"]
+
+    # Verifies library is not compatible
+    assert data[1]["library"]["name"] == "wm8978-esp32"
+    assert "compatible" not in data[1]["library"]
+
+
 def test_install(run_command):
     libs = ['"AzureIoTProtocol_MQTT"', '"CMMC MQTT Connector"', '"WiFiNINA"']
     # Should be safe to run install multiple times