Even when no libraries are used, precompiled ones must be removed

Signed-off-by: Federico Fissore <[email protected]>

Author: Federico Fissore

Diff for: src/arduino.cc/builder/test/unused_compiled_libraries_remover_test.go

@@ -75,3 +75,30 @@ func TestUnusedCompiledLibrariesRemoverLibDoesNotExist(t *testing.T) {
 	err := cmd.Run(context)
 	NoError(t, err)
 }
+
+func TestUnusedCompiledLibrariesRemoverNoUsedLibraries(t *testing.T) {
+	temp, err := ioutil.TempDir("", "test")
+	NoError(t, err)
+	defer os.RemoveAll(temp)
+
+	NoError(t, os.MkdirAll(filepath.Join(temp, "SPI"), os.FileMode(0755)))
+	NoError(t, os.MkdirAll(filepath.Join(temp, "Bridge"), os.FileMode(0755)))
+	NoError(t, ioutil.WriteFile(filepath.Join(temp, "dummy_file"), []byte{}, os.FileMode(0644)))
+
+	context := make(map[string]interface{})
+	context[constants.CTX_LIBRARIES_BUILD_PATH] = temp
+	context[constants.CTX_IMPORTED_LIBRARIES] = []*types.Library{}
+
+	cmd := builder.UnusedCompiledLibrariesRemover{}
+	err = cmd.Run(context)
+	NoError(t, err)
+
+	_, err = os.Stat(filepath.Join(temp, "SPI"))
+	require.Error(t, err)
+	require.True(t, os.IsNotExist(err))
+	_, err = os.Stat(filepath.Join(temp, "Bridge"))
+	require.Error(t, err)
+	require.True(t, os.IsNotExist(err))
+	_, err = os.Stat(filepath.Join(temp, "dummy_file"))
+	NoError(t, err)
+}

Diff for: src/arduino.cc/builder/unused_compiled_libraries_remover.go

@@ -44,10 +44,6 @@ func (s *UnusedCompiledLibrariesRemover) Run(context map[string]interface{}) err
 	librariesBuildPath := context[constants.CTX_LIBRARIES_BUILD_PATH].(string)
 	libraries := context[constants.CTX_IMPORTED_LIBRARIES].([]*types.Library)
 
-	if len(libraries) == 0 {
-		return nil
-	}
-
 	_, err := os.Stat(librariesBuildPath)
 	if err != nil && os.IsNotExist(err) {
 		return nil