Build issues

[perlinm]

I am not 100% sure that this is a relay-bp issue, but I have a github workflow that installs relay-bp==0.1.0, and fails with the following error:

      💥 maturin failed
        Caused by: python-source is set to
      `/home/runner/work/_temp/setup-uv-cache/sdists-v9/pypi/relay-bp/0.1.0/ik2kgaSBgG6dv5O6CNc-n/src/src`,
      but the python module at
      `/home/runner/work/_temp/setup-uv-cache/sdists-v9/pypi/relay-bp/0.1.0/ik2kgaSBgG6dv5O6CNc-n/src/src/_relay_bp`
      does not exist. Either create the Python module or remove the
      `python-source` setting from pyproject.toml.
      Error: command ['maturin', 'pep517', 'build-wheel', '-i',
      '/home/runner/work/_temp/setup-uv-cache/builds-v0/.tmpR12HWT/bin/python',
      '--compatibility', 'off'] returned non-zero exit status 1

      hint: This usually indicates a problem with the package or the build
      environment.

Any ideas what the problem might be, or how to fix it?

[perlinm]

Looks like if I clone the repo and try a maturin build --sdist I get the following error:

$ maturin build --sdist
📦 Including license file `LICENSE.txt`
💥 maturin failed
  Caused by: python-source is set to `<REPO_PARENT_DIR>/relay/src`, but the python module at `<REPO_PARENT_DIR>/relay/src/_relay_bp` does not exist. Either create the Python module or remove the `python-source` setting from pyproject.toml.

But this is fixed if I change this line in crates/relay_bp_py/Cargo.toml from name = "_relay_bp" to name = "relay_bp", in which case maturin build --sdist seems to build a wheel.

Maybe this is the issue? Although I still get the following warning from running maturin build --sdist:

⚠️  Warning: Couldn't find the symbol `PyInit_relay_bp` in the native library. Python will fail to import this module. If you're using pyo3, check that `#[pymodule]` uses `relay_bp` as module name

[perlinm]

Nevermind, that doesn't work, because it breaks various from _relay_bp import _bp commands in the repo.

[trmue]

Hi! We've been observing something similar, and it seems to be related to a recent Maturin update
https://www.maturin.rs/changelog.html

1.12.0 was released on the 14th of February. 1.12.1 was released earlier today (16th February) and supposedly has build fix.
I can't say right now whether this actually fixes the issue, or whether there is something that needs to be fixed about the build setup in this repo.

[trmue]

The quick fix seems to be to use Maturin 1.11.5. I will monitor which updates they push and whether changes will be required.

[trmue]

I think PR #30 should fix this. Can you give this a try, @perlinm ?

[perlinm]

Thanks for the quick response! Looks like the fix in #30 works (there are still some tests failing, but I believe for unrelated reasons).