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

Make Ammonite cache its files in TMPDIR. #1160

Merged
merged 1 commit into from
Dec 31, 2024
Merged

Conversation

gouttegd
Copy link
Contributor

When running a Scala script, Ammonite attempts to download and store the script's dependencies in /tools/.coursier-cache, which requires root privileges.

We force Ammonite to write its cache in the current ontology's TMPDIR instead.

closes #1159

When running a Scala script, Ammonite attempts to download and store the
script's dependencies in /tools/.coursier-cache, which requires root
privileges.

We force Ammonite to write its cache in the current ontology's TMPDIR
instead.

closes #1159
@gouttegd gouttegd self-assigned this Dec 28, 2024
@gouttegd gouttegd requested a review from matentzn December 28, 2024 18:12
Copy link
Contributor

@matentzn matentzn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, have not tested it!

@gouttegd
Copy link
Contributor Author

I have. :)

But this fix is only a stop-gap measure. Ultimately, I want to replace the validate_id_ranges.sc script by a better tool to manage the ID range file. I’ve started working on that here.

@gouttegd gouttegd merged commit c31beae into master Dec 31, 2024
1 check passed
@gouttegd gouttegd deleted the fix-amm-invocation branch December 31, 2024 11:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

validate_idranges target is broken
2 participants