Skip to content

Clean up dependencies installation symlinks on exit #29

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

Merged
merged 1 commit into from
Aug 1, 2021
Merged

Clean up dependencies installation symlinks on exit #29

merged 1 commit into from
Aug 1, 2021

Conversation

per1234
Copy link
Collaborator

@per1234 per1234 commented Aug 1, 2021

The code used to install Arduino CLI, platform, and library dependencies does so by creating symlinks in the appropriate locations on the runner. The files the script gathers via clone or download, are stored in a temporary folder that is deleted when the script exits. This means the installation symlink for those files has no possible use after the script finishes.

This particular pollution of the runner's environment is resolved by also registering a call to delete on exit as each symlink is created.

The code used to install Arduino CLI, platform, and library dependencies does this by creating symlinks in the
appropriate locations on the runner. The files the script gathers via clone or download, are stored in a temporary folder
that is deleted when the script exits. This means the installation symlink for those files has no possible use after the
script finishes.

This particular pollution of the runner's environment is resolved by also registering a call to delete on exit as each
symlink is created.
@per1234 per1234 added type: imperfection Perceived defect in any part of project topic: code Related to content of the project itself labels Aug 1, 2021
@codecov-commenter
Copy link

Codecov Report

Merging #29 (083486c) into main (c6c5547) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##              main       #29   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            2         2           
  Lines         1596      1598    +2     
=========================================
+ Hits          1596      1598    +2     
Impacted Files Coverage Δ
compilesketches/compilesketches.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c6c5547...083486c. Read the comment docs.

@per1234 per1234 merged commit 9fbb6d0 into arduino:main Aug 1, 2021
@per1234 per1234 deleted the clean-installation-symlinks branch August 1, 2021 01:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: code Related to content of the project itself type: imperfection Perceived defect in any part of project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants