Skip to content

Conversation

theroggy
Copy link
Member

@theroggy theroggy commented Jul 5, 2025

This PR:

resolves #378

@dg0yt
Copy link

dg0yt commented Jul 5, 2025

Would it help to include libspatialite without freexl?

@theroggy
Copy link
Member Author

theroggy commented Jul 5, 2025

Would it help to include libspatialite without freexl?

Yes, good idea: freexl is not needed for our use!

I'll give it a try.

@dg0yt
Copy link

dg0yt commented Jul 5, 2025

A little bit different. Add libspatialite as a top-level dependency, with default-features: false.

@dg0yt
Copy link

dg0yt commented Jul 6, 2025

vcpkg linux logs are still reported to be in /opt/vcpkg/buildtrees - this is not what is uploaded in the next step.

@theroggy
Copy link
Member Author

theroggy commented Jul 7, 2025

vcpkg linux logs are still reported to be in /opt/vcpkg/buildtrees - this is not what is uploaded in the next step.

Thanks for having a look! I'm not a specialist at all in this, so much appreciated!

I only made the change in one of the 3 linux containers/builds (manylinux_2_28_x86_64-vcpkg-gdal), so sorry for the confusion. I now changed it in all three...

But, the log files are not found anyway...

I wondered if it has something to do with paths in the container versus local paths, so I tried copying the files first to a local path,, but I don't seem to get that working either :-(...

@theroggy theroggy changed the title BUILD: quick check if LibKML and LibSpatialite can be included now BUILD: quick check if LibSpatialite can be included now Sep 25, 2025
@theroggy theroggy changed the title BUILD: quick check if LibSpatialite can be included now BUILD: add LibSpatialite to wheels Sep 25, 2025
@theroggy theroggy changed the title BUILD: add LibSpatialite to wheels BLD/REL: add LibSpatialite to wheels Sep 26, 2025
@theroggy theroggy marked this pull request as ready for review September 26, 2025 09:24
@theroggy theroggy added this to the 0.12.0 milestone Sep 26, 2025
@theroggy theroggy changed the title BLD/REL: add LibSpatialite to wheels BLD/REL: add libspatialite to the wheels Sep 27, 2025
@jorisvandenbossche
Copy link
Member

@theroggy this seems to "just work" now?

@theroggy
Copy link
Member Author

theroggy commented Oct 14, 2025

@theroggy this seems to "just work" now?

Yes, indeed... something must have been fixed upstream...

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.

BLD/RLS: to include spatialite in wheels or not to?

3 participants