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

maturin now requires version in pyproject.toml #560

Open
bluegenes opened this issue Dec 29, 2024 · 2 comments
Open

maturin now requires version in pyproject.toml #560

bluegenes opened this issue Dec 29, 2024 · 2 comments

Comments

@bluegenes
Copy link
Contributor

pip install -e .                           

Obtaining file:///Users/ntward/dib-lab/sourmash_plugin_branchwater
  Installing build dependencies ... done
  Checking if build backend supports build_editable ... done
  Getting requirements to build editable ... done
  Preparing editable metadata (pyproject.toml) ... error
  error: subprocess-exited-with-error
  
  × Preparing editable metadata (pyproject.toml) did not run successfully.
  │ exit code: 1
  ╰─> [5 lines of output]
      💥 maturin failed
        Caused by: `project.version` field is required in pyproject.toml unless it is present in the `project.dynamic` list
      Error running maturin: Command '['maturin', 'pep517', 'write-dist-info', '--metadata-directory', '/private/var/folders/pv/_f7m99kd33782sdc79lfqc300000gn/T/pip-modern-metadata-vl8u3z9a', '--interpreter', '/Users/ntward/miniforge3/envs/branchwater-dev/bin/python3.12']' returned non-zero exit status 1.
      Checking for Rust toolchain....
      Running `maturin pep517 write-dist-info --metadata-directory /private/var/folders/pv/_f7m99kd33782sdc79lfqc300000gn/T/pip-modern-metadata-vl8u3z9a --interpreter /Users/ntward/miniforge3/envs/branchwater-dev/bin/python3.12`
      [end of output]
  
  note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed

× Encountered error while generating package metadata.
╰─> See above for output.

note: This is an issue with the package mentioned above, not pip.
hint: See above for details.
@ctb
Copy link
Collaborator

ctb commented Dec 29, 2024

weird. I think this must be new...

@ctb
Copy link
Collaborator

ctb commented Dec 29, 2024

ahh, found it. Seems to be required as of 1.80.

PyO3/maturin#2391 and linked issues, e.g. FrankC01/pysui-fastcrypto#9

Wonder if we could use dynamic? Let's leave this open for now, but I will approve the merge ;)

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

No branches or pull requests

2 participants