-
-
Notifications
You must be signed in to change notification settings - Fork 491
Commit 125bd64
Install Arduino CLI build dependencies in all dependent workflows
Arduino CLI is a tool dependency of Arduino IDE. For this reason, the necessary Arduino CLI build is acquired whenever
running the `yarn` command in the repository.
The way the Arduino CLI build is acquired depends on the type of version specified as dependency in the
`arduino.cli.version` field of the arduino-ide-extension package metadata:
- Release/nightly: download pre-built standard distribution
- Git ref: build from source
This means that, in the latter case, all build dependencies of Arduino CLI must be present. While the Go module
dependencies are automatically installed during the build, the build tool dependencies must be installed in advance:
- Go programming language
- Task task runner
Arduino IDE's infrastructure was recently changed to use the Task tool to build Arduino CLI in the supported manner. A
step to install Task was not added to some of the workflows that run `yarn`, which caused them to fail when a
non-release version of Arduino CLI was used as a dependency:
arduino-ide-extension: >>> Building the CLI...
arduino-ide-extension: /bin/sh: 1: task: not found
arduino-ide-extension: error Command failed with exit code 1.
A step for the missing tool dependency is hereby added to those workflows.
The lack of an explicit installation of the other dependency, Go did not result in an error because Go is pre-installed
on the GitHub Actions runner. However, the installed version may not match the version Arduino CLI is intended to be
built with and validated for, and the version provided by the runner may change at any time. For this reason, it will be
safest to explicitly set up the appropriate version of Go in the workflows.1 parent ca47e8a commit 125bd64
File tree
5 files changed
+59
-0
lines changed- .github/workflows
5 files changed
+59
-0
lines changedOriginal file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
28 | 28 |
| |
29 | 29 |
| |
30 | 30 |
| |
31 | + | ||
32 | + | ||
31 | 33 |
| |
32 | 34 |
| |
33 | 35 |
| |
| |||
66 | 68 |
| |
67 | 69 |
| |
68 | 70 |
| |
71 | + | ||
72 | + | ||
73 | + | ||
74 | + | ||
75 | + | ||
69 | 76 |
| |
70 | 77 |
| |
71 | 78 |
| |
|
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1 | 1 |
| |
2 | 2 |
| |
3 | + | ||
4 | + | ||
5 | + | ||
6 | + | ||
3 | 7 |
| |
4 | 8 |
| |
5 | 9 |
| |
| |||
31 | 35 |
| |
32 | 36 |
| |
33 | 37 |
| |
38 | + | ||
39 | + | ||
40 | + | ||
41 | + | ||
42 | + | ||
34 | 43 |
| |
35 | 44 |
| |
36 | 45 |
| |
|
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1 | 1 |
| |
2 | 2 |
| |
3 | + | ||
4 | + | ||
5 | + | ||
6 | + | ||
3 | 7 |
| |
4 | 8 |
| |
5 | 9 |
| |
| |||
18 | 22 |
| |
19 | 23 |
| |
20 | 24 |
| |
25 | + | ||
26 | + | ||
27 | + | ||
28 | + | ||
29 | + | ||
30 | + | ||
31 | + | ||
32 | + | ||
33 | + | ||
34 | + | ||
35 | + | ||
21 | 36 |
| |
22 | 37 |
| |
23 | 38 |
| |
|
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1 | 1 |
| |
2 | 2 |
| |
3 | + | ||
4 | + | ||
5 | + | ||
6 | + | ||
3 | 7 |
| |
4 | 8 |
| |
5 | 9 |
| |
| |||
18 | 22 |
| |
19 | 23 |
| |
20 | 24 |
| |
25 | + | ||
26 | + | ||
27 | + | ||
28 | + | ||
29 | + | ||
30 | + | ||
31 | + | ||
32 | + | ||
33 | + | ||
34 | + | ||
35 | + | ||
21 | 36 |
| |
22 | 37 |
| |
23 | 38 |
| |
|
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
7 | 7 |
| |
8 | 8 |
| |
9 | 9 |
| |
10 | + | ||
11 | + | ||
10 | 12 |
| |
11 | 13 |
| |
12 | 14 |
| |
| |||
22 | 24 |
| |
23 | 25 |
| |
24 | 26 |
| |
27 | + | ||
28 | + | ||
29 | + | ||
30 | + | ||
31 | + | ||
32 | + | ||
33 | + | ||
34 | + | ||
35 | + | ||
36 | + | ||
37 | + | ||
25 | 38 |
| |
26 | 39 |
| |
27 | 40 |
| |
|
0 commit comments