Skip to content

Conversation

@virtuald
Copy link
Member

@virtuald virtuald commented Feb 26, 2025

Supercedes #26, but doesn't work yet. I think that shapely doesn't support cross-compilation because of the way that it detects numpy as a dependency.

shapely depends on libgeos, and what this does is adds a separate step that builds libgeos and installs it into the sysroot (but only with static linking), so in theory if the wheel gets built it should just work. However it fails when trying to link to some numpy stuff.

I've brought this as far as I want to go with it, someone else will need to fix shapely.

@virtuald virtuald mentioned this pull request Feb 26, 2025
@virtuald virtuald force-pushed the shapely branch 3 times, most recently from 6a8667b to b930d12 Compare February 26, 2025 16:39
@JayTicku
Copy link
Contributor

I think your errors are not coming from numpy but from Cython. Here is a GitHub issue that describes the problem that the checks are returning and a solution: cython/cython#6249

@JayTicku
Copy link
Contributor

JayTicku commented Feb 26, 2025

Would there be a way to force it to use numpy 2.0.0 even though 2.2.1 is installed, as described as one possible solution to the Cython issue?

@virtuald
Copy link
Member Author

Yeah, I had seen that, and had tried a variety of permutations of versions. It seems like the unreleased version of shapely might have fixes for this stuff?

I'm done messing with it for now. If you can find a combination of things that works locally, post it and we can update this.

@JayTicku
Copy link
Contributor

Thanks for your help. I found other package that can accomplish my use case that's written in pure python. It has similar features and is a lighter version of Shapely so I don't think I will need Shapely anymore.

@virtuald virtuald merged commit 695afd9 into 2025 Mar 1, 2025
20 of 22 checks passed
@virtuald virtuald deleted the shapely branch March 2, 2025 04:42
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