Skip to content

fix(ci): Fix libgpuspatial version spec for wheels workflow#769

Merged
paleolimbot merged 2 commits intoapache:mainfrom
paleolimbot:fix-wheels-workflow
Apr 17, 2026
Merged

fix(ci): Fix libgpuspatial version spec for wheels workflow#769
paleolimbot merged 2 commits intoapache:mainfrom
paleolimbot:fix-wheels-workflow

Conversation

@paleolimbot
Copy link
Copy Markdown
Member

The wheels workflow was failing:

  cargo 1.95.0 (f2d3ce0bd 2026-03-21)
  Running `maturin pep517 build-wheel -i /tmp/build-env-1p9hnzh9/bin/python --compatibility off --features s2geography,pyo3/extension-module`
      Updating crates.io index
  error: failed to select a version for the requirement `sedona-libgpuspatial = "^0.4.0"`
  candidate versions found which didn't match: 0.4.0-alpha52
  location searched: /project/c/sedona-libgpuspatial
  required by package `sedona-spatial-join-gpu v0.4.0-alpha52 (/project/rust/sedona-spatial-join-gpu)`
  if you are looking for the prerelease package it needs to be specified explicitly
      sedona-libgpuspatial = { version = "0.4.0-alpha52" }
  💥 maturin failed
    Caused by: Cargo metadata failed. Does your crate compile with `cargo build`?
    Caused by: `cargo metadata` exited with an error: 
  Error: command ['maturin', 'pep517', 'build-wheel', '-i', '/tmp/build-env-1p9hnzh9/bin/python', '--compatibility', 'off', '--features', 's2geography,pyo3/extension-module'] returned non-zero exit status 1

This was becase set_dev_version.py has a very specifically defined regex that expected a single space where there were two. I've made it back into a single space but also tried to make the regex a little more lenient.

Briefly, we do this so that we can set a dev version so that when we upload the dev build wheels it looks like a new version.

@github-actions github-actions bot requested a review from zhangfengcdt April 16, 2026 22:07
@paleolimbot paleolimbot marked this pull request as ready for review April 16, 2026 22:37
@paleolimbot paleolimbot merged commit 326212c into apache:main Apr 17, 2026
17 checks passed
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.

2 participants