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

Fix missing PyInit_<module_name> warning with Rust submodule #528

Merged
merged 2 commits into from
May 6, 2021

Conversation

messense
Copy link
Member

@messense messense commented May 6, 2021

Fixes an oversight of #489 , it doesn't cause any runtime issue but the warning is annoying.

⚠  Warning: Couldn't find the symbol `PyInit_pyo3_mixed_submodule` in the native library. Python will fail to import this module. If you're using pyo3, check that `#[pymodule]` uses `pyo3_mixed_submodule` as module name
📦 Built wheel for CPython 3.8 to /io/target/wheels/pyo3_mixed_submodule-2.1.3-cp38-cp38-manylinux_2_5_x86_64.manylinux1_x86_64.whl

https://github.com/PyO3/maturin/runs/2518736555

Copy link
Member

@konstin konstin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

src/compile.rs Outdated Show resolved Hide resolved
@messense messense force-pushed the warn-missing-py-init branch 2 times, most recently from ee6ca7b to ea8730a Compare May 6, 2021 15:53
@messense messense merged commit 6d051af into PyO3:main May 6, 2021
@messense messense deleted the warn-missing-py-init branch May 6, 2021 16:13
@messense messense mentioned this pull request May 7, 2021
bmwiedemann pushed a commit to bmwiedemann/openSUSE that referenced this pull request May 22, 2021
https://build.opensuse.org/request/show/894986
by user mia + dimstar_suse
- Update to 0.10.6
  * Fix build with the “upload” feature disabled by ravenexp
    gh#PyO3/maturin#548
- Changes in 0.10.5:
  * Add manylinux_2_27 support
    gh#PyO3/maturin#521
  * Add support for Windows arm64 target
    gh#PyO3/maturin#524
  * Always output PEP 600 platform tags
    gh#PyO3/maturin#525
  * Fix missing PyInit_<module_name> warning with Rust submodule
    gh#PyO3/maturin#528
  * Better cross compiling support for PyO3 binding on Unix
    gh#PyO3/maturin#454
  * Fix s390x architecture support
    gh#PyO3/maturin#530
  * Fix auditwheel panic with s390x wheels
    gh#PyO3/maturin#532
  * Support uploading heterogenous wheels by ravenexp
    gh#PyO3/maturin#544
  * Warn about pyproject.toml missing maturin version constraint
    gh#PyO3/maturin#545
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.

None yet

2 participants