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

Update to use jll artifact #10

Merged
merged 5 commits into from Apr 20, 2020
Merged

Update to use jll artifact #10

merged 5 commits into from Apr 20, 2020

Conversation

jaakkor2
Copy link
Contributor

Copy link
Member

@visr visr left a comment

Choose a reason for hiding this comment

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

This is great, thanks! This is the last build in JuliaGeo to make the switch. Left a few minor comments. Could you also add a line to the readme saying that libspatialindex builds are provided, linking to both https://github.com/JuliaPackaging/Yggdrasil/tree/master/L/LibSpatialIndex and https://github.com/JuliaBinaryWrappers/LibSpatialIndex_jll.jl?

@MaximeBouton are you ok with merging this, and archiving https://github.com/JuliaGeo/LibSpatialIndexBuilder?

.travis.yml Show resolved Hide resolved
Project.toml Show resolved Hide resolved
add [compat] entry for LibSpatialIndex_jll
@jaakkor2 jaakkor2 requested a review from visr April 18, 2020 10:15
@MaximeBouton
Copy link
Contributor

This is great, thanks!
@visr I am fine with archiving the other repo.

@visr visr merged commit 7418122 into JuliaGeo:master Apr 20, 2020
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.

None yet

3 participants