Skip to content

Commit

Permalink
chore: update to maturin's recommended project layout for rust/python…
Browse files Browse the repository at this point in the history
… projects

The previous layout leads to an import error when installing with `maturin build` and `pip install .`.

This error was common enough that `maturin` changed the recommended project layout to what this commit does.

A prior PR attempted to solve this by altering `lib.name` in Cargo.toml, but that did not work for me.

- [Prior PR](apache#694)
- [maturin ImportError issue](PyO3/maturin#490)
- [maturin changes recommended project structure](PyO3/maturin#855)
  • Loading branch information
Michael-J-Ward committed May 14, 2024
1 parent 01a370e commit 0b47100
Show file tree
Hide file tree
Showing 28 changed files with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -55,6 +55,7 @@ repository = "https://github.com/apache/arrow-datafusion-python"
profile = "black"

[tool.maturin]
python-source = "python"
module-name = "datafusion._internal"
include = [
{ path = "Cargo.lock", format = "sdist" }
Expand Down
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.

0 comments on commit 0b47100

Please sign in to comment.