Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use dedicated location for "built-in" libraries #1055

Closed
3 tasks done
per1234 opened this issue Jun 13, 2022 · 2 comments · Fixed by #1370
Closed
3 tasks done

Use dedicated location for "built-in" libraries #1055

per1234 opened this issue Jun 13, 2022 · 2 comments · Fixed by #1370
Assignees
Labels
conclusion: resolved Issue was resolved topic: CLI Related to Arduino CLI topic: code Related to content of the project itself type: enhancement Proposed improvement

Comments

@per1234
Copy link
Contributor

per1234 commented Jun 13, 2022

Describe the request

Use a dedicated location for the "built-in" library installations, with a lower location priority than platform bundled libraries.

🙂 Library dependency resolution will match the behavior of Arduino IDE 1.x

Describe the current behavior

In order to allow users to get started with Arduino with the minimum amount of complexity, the Arduino IDE 1.x installation comes with a collection of fundamental Arduino libraries (e.g., Keyboard, Mouse, Servo) pre-installed. Even experienced users now expect these libraries to be available by default, and many tutorials (both official and 3rd party) have been written based on that assumption. So it is important that Arduino IDE 2.x also provide these libraries.

In order to simplify the Arduino IDE packaging process and ensure users are provided with the newest available, the Arduino Library Manager infrastructure is currently used to install the libraries the first time the Arduino IDE is started (#663).

Library Manager installs these libraries to the sketchbook folder. Libraries at this location have a higher "location priority" than they did when bundled with the Arduino IDE installation.

🙁 This priority increase can cause the "built-in" library to now be selected by the Arduino sketch build process in a case where a platform bundled library would have been selected when using Arduino IDE 1.x.

Arduino IDE version

2.0.0-rc7

Operating system

All

Operating system version

N/A

Additional context

This would also provide an alternative solution to #798

Issue checklist

  • I searched for previous requests in the issue tracker
  • I verified the feature was still missing when using the latest nightly build
  • My request contains all necessary details
@per1234 per1234 added type: enhancement Proposed improvement topic: code Related to content of the project itself labels Jun 13, 2022
@ubidefeo
Copy link

@per1234
it seems we thought we were solving an issue by bundling the legacy libraries but we caused another one 🤔

@kittaakos
Copy link
Contributor

Upstream PR: arduino/arduino-cli#1817

kittaakos pushed a commit that referenced this issue Aug 30, 2022
Closes #1055.

Signed-off-by: Akos Kitta <a.kitta@arduino.cc>
kittaakos pushed a commit that referenced this issue Aug 31, 2022
Closes #1055.

Signed-off-by: Akos Kitta <a.kitta@arduino.cc>
kittaakos pushed a commit that referenced this issue Sep 2, 2022
Closes #1055.

Signed-off-by: Akos Kitta <a.kitta@arduino.cc>
@per1234 per1234 added the conclusion: resolved Issue was resolved label Sep 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
conclusion: resolved Issue was resolved topic: CLI Related to Arduino CLI topic: code Related to content of the project itself type: enhancement Proposed improvement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants