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 copy_test_packages in some more places, and ensure that the copied test package is user writable. #4136

Merged
merged 1 commit into from
Jan 16, 2025

Conversation

rbvermaa
Copy link
Contributor

@rbvermaa rbvermaa commented Jan 14, 2025

The tests assume the copied files are writable, so this makes this assumption explicit and enforces it. This allows running the Pkg tests as part of Base.runtests of an installed Julia installation, where sources might be readonly.

@rbvermaa rbvermaa force-pushed the rv-copy-test-package branch 3 times, most recently from a538db9 to 8064751 Compare January 14, 2025 15:03
…t package is user writable. This allows running the Pkg tests as part of Base.runtests of an installed Julia installation, where sources might be readonly.
@rbvermaa rbvermaa force-pushed the rv-copy-test-package branch from 8064751 to 7e7599b Compare January 14, 2025 21:03
@rbvermaa rbvermaa changed the title Use copy_test_packages in some places, and ensure that the copied test package is user writable. Use copy_test_packages in some more places, and ensure that the copied test package is user writable. Jan 15, 2025
@rbvermaa rbvermaa marked this pull request as ready for review January 15, 2025 08:25
@KristofferC KristofferC merged commit df1931a into JuliaLang:master Jan 16, 2025
9 checks passed
@rbvermaa rbvermaa deleted the rv-copy-test-package branch January 22, 2025 12:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants