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

Add geography extension #303

Merged
merged 2 commits into from
Feb 21, 2025
Merged

Add geography extension #303

merged 2 commits into from
Feb 21, 2025

Conversation

paleolimbot
Copy link
Contributor

Hi! I've been working on a 'geography' extension to complement 'spatial' for data on the sphere. I think it's ready for community extension status, but let me know if there are missing components here that need to be handled!

@paleolimbot
Copy link
Contributor Author

cc @Maxxen!

@paleolimbot
Copy link
Contributor Author

The error seems to be :

 error: Invalid triplet: x64-windows-static-md-release

I believe this build is passing on duckdb-geography CI ( https://github.com/paleolimbot/duckdb-geography ), but let me know if something is misconfigured on my end!

@rustyconover
Copy link
Contributor

Hi @paleolimbot,

I see the issue—x64-windows-static-md-release was introduced in vcpkg after the version currently used in the build (see git blame).

This happened because the community-extensions repo’s default vcpkg commit wasn’t updated alongside extension-ci-tools for DuckDB 1.2.0.

I've opened a PR to fix this: #304.

Cheers,
Rusty

@carlopi
Copy link
Collaborator

carlopi commented Feb 21, 2025

@paleolimbot: I will come back on your repo vs wasm builds, removing them also had the nice side-effect of retriggering CI.

@rustyconover: thanks for the fix, windows build is now going further, great!

@carlopi carlopi merged commit 7dd4fef into duckdb:main Feb 21, 2025
31 checks passed
@carlopi
Copy link
Collaborator

carlopi commented Feb 21, 2025

Thanks and welcome!

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.

3 participants