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 Scratch.jl for update messages #246

Open
Datseris opened this issue Apr 28, 2021 · 2 comments
Open

Use Scratch.jl for update messages #246

Datseris opened this issue Apr 28, 2021 · 2 comments
Labels
low priority Not super important.

Comments

@Datseris
Copy link
Member

From a discussion on Slack that will be forever lost, it is better to use https://github.com/JuliaPackaging/Scratch.jl to display update messages instead of the current pipeline with touch. Stefan Karpinski said that

We really don’t want people modifying package code in place 😢

and also cited other related problems, while Elliot Saba further commended that the approach will be impossible in the future:

We’re eventually going to make packages read-only.

Elliot thankfully pointed out that the replacement is easy:

https://github.com/JuliaPackaging/Scratch.jl , Basically, instead of using the DIR macro, you would use the get_scratch macro, that's the only change. But you get a lot of benefits, including that it will work even if your package is included in PackageCompiler output. The scratch directory can be cleared, if for example the package is uninstalled then reinstalled later. The user can also manually clear it.

@Datseris Datseris added the low priority Not super important. label Apr 28, 2021
@Datseris
Copy link
Member Author

@Libbum keep this in mind as well for Agents.jl

@Libbum
Copy link
Member

Libbum commented Apr 28, 2021

Cool. This is a decent change. Will add it to the list—thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
low priority Not super important.
Projects
None yet
Development

No branches or pull requests

2 participants