Skip to content

Commit 46e9273

Browse files
committed
bugfix: monitorDependencies are now installed with the core
1 parent 23e7cb5 commit 46e9273

File tree

1 file changed

+11
-2
lines changed

1 file changed

+11
-2
lines changed

arduino/cores/packagemanager/download.go

+11-2
Original file line numberDiff line numberDiff line change
@@ -96,14 +96,23 @@ func (pm *PackageManager) FindPlatformReleaseDependencies(item *PlatformReferenc
9696
return nil, nil, fmt.Errorf(tr("getting tool dependencies for platform %[1]s: %[2]s"), release.String(), err)
9797
}
9898

99-
// discovery dependencies differ from normal tool since we always want to use the latest \
99+
// discovery dependencies differ from normal tool since we always want to use the latest
100100
// available version for the platform package
101101
discoveryDependencies, err := pm.Packages.GetPlatformReleaseDiscoveryDependencies(release)
102102
if err != nil {
103103
return nil, nil, fmt.Errorf(tr("getting discovery dependencies for platform %[1]s: %[2]s"), release.String(), err)
104104
}
105+
toolDeps = append(toolDeps, discoveryDependencies...)
105106

106-
return release, append(toolDeps, discoveryDependencies...), nil
107+
// monitor dependencies differ from normal tool since we always want to use the latest
108+
// available version for the platform package
109+
monitorDependencies, err := pm.Packages.GetPlatformReleaseMonitorDependencies(release)
110+
if err != nil {
111+
return nil, nil, fmt.Errorf(tr("getting monitor dependencies for platform %[1]s: %[2]s"), release.String(), err)
112+
}
113+
toolDeps = append(toolDeps, monitorDependencies...)
114+
115+
return release, toolDeps, nil
107116
}
108117

109118
// DownloadToolRelease downloads a ToolRelease. If the tool is already downloaded a nil Downloader

0 commit comments

Comments
 (0)