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

Simplify dependencies #171

Closed
rafaqz opened this issue Aug 20, 2024 · 3 comments
Closed

Simplify dependencies #171

rafaqz opened this issue Aug 20, 2024 · 3 comments

Comments

@rafaqz
Copy link

rafaqz commented Aug 20, 2024

I have been asked if it makes sense to add this as a dependency in much of the geospatial data ecosystem.

I think it does, but it seems that this package has a few too many dependencies to be a low-level dependency of other packages? Now that these base standard libs are not in the system image these package have real costs.

It seems to me that:

  • MMap.jl is can be in an extension (its just one method?)
  • UUIDs.jl is it really needed? Base.tempname doesn't use it.
  • Printf.jl seems to be needed for one call to @sprintf - can we just use sprint ?
  • Test.jl can be in an extension

CC @asinghvi17

@rofinn
Copy link
Owner

rofinn commented Aug 22, 2024

Interesting, I haven't been paying close attention to Julia lately and didn't realize they were removed from the sysimg. I'm not sure I agree with the decision, but if it's easier to define extensions now then I'm fine to split up that functionality. My understanding is that Requires.jl isn't needed anymore? When I used that for FilePaths.jl it came with significant overhead.

@rafaqz
Copy link
Author

rafaqz commented Aug 23, 2024

Yes, extensions are very easy and built in now, with none of the Requires.jl downsides. Especially including Test as a dependency is something I've seen complaints about recently, and everyone is switching to putting it in an extension.

@rofinn
Copy link
Owner

rofinn commented Aug 27, 2024

I think this is good to close now. If we should try to isolate dates as well, feel free to open a separate issue.

@rofinn rofinn closed this as completed Aug 27, 2024
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

No branches or pull requests

2 participants