Description
Describe the bug
Test builds are generated for every PR creation and push as well as pushes to the main
branch. These make it easy for anyone to contribute to the project through beta testing.
It is important that the specific test build in use be easily identifiable. The Git commit hash is used for this purpose. For example, the version assigned to this commit: 9ecff86 is 2.0.0-rc3-snapshot-9ecff86
🐛 The version indicator is now added twice to the test build versions. For example, the the version assigned to this commit: 9b1f15d is 2.0.0-rc4-snapshot.9b1f15d-snapshot-9b1f15d
To Reproduce
- Open the list of
push
orpull_request
event-triggered runs of theArduino IDE
workflow:
https://github.com/arduino/arduino-ide/actions/workflows/build.yml?query=event%3Apull_request+event%3Apush+is%3Asuccess - Click on any recent item in the list of runs.
For example: https://github.com/arduino/arduino-ide/actions/runs/1858133816 - Under the "Artifacts" section of the page, click on the download link of the artifact for your operating system.
- Wait for the download to finish.
- Extract or install the downloaded file.
🐛 The extracted folder contains redundant version indicators (e.g.,arduino-ide_2.0.0-rc4-snapshot.9b1f15d-snapshot-9b1f15d_Windows_64bit
) - Start the Arduino IDE test build.
🐛 The window title contains redundant version indicators (e.g., "sketch_feb17a | Arduino IDE 2.0.0-rc4-snapshot.9b1f15d-snapshot-9b1f15d") - Select Help > About Arduino IDE from the Arduino IDE menus.
- Click the Copy button.
- Paste the contents of the clipboard into any text editor.
🐛 The pasted text contains redundant version indicators.
For example:Version: 2.0.0-rc4-snapshot.9b1f15d-snapshot-9b1f15d Date: 2022-02-17T10:40:50.003Z CLI Version: 0.21.0 [10107d24] Copyright © 2022 Arduino SA
Expected behavior
The tester builds don't have redundant version indicators (e.g., 2.0.0-rc4-snapshot.9b1f15d
).
Desktop
- OS: Windows 10, Ubuntu 20.04
- Version: 2.0.0-rc4-snapshot.9b1f15d-snapshot-9b1f15d
Date: 2022-02-17T10:40:50.003Z
CLI Version: 0.21.0 [10107d24]
Additional context
The change was introduced at f660058
There was a previous change from the snapshot.<hash>
(e.g., 2.0.0-rc3-snapshot.69ac1f4
) to snapshot-<hash>
format (e.g., 2.0.0-rc3-snapshot-112153f
), which was introduced in 112153f
The additional characters use up some of the precious allowance of path length doled out by Microsoft for the beta testers on Windows.
Seeing this identifier in the filename redundantly, but with two different formats (note the .
in one and -
in the other) makes me nervous that it is an indication of something fragile.