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

Configure bors #1214

Merged
merged 7 commits into from
Nov 1, 2022
Merged

Configure bors #1214

merged 7 commits into from
Nov 1, 2022

Conversation

messense
Copy link
Member

No description provided.

@netlify
Copy link

netlify bot commented Oct 26, 2022

Deploy Preview for maturin-guide ready!

Name Link
🔨 Latest commit e477a9e
🔍 Latest deploy log https://app.netlify.com/sites/maturin-guide/deploys/6360950bf0fc49000833a2de
😎 Deploy Preview https://deploy-preview-1214--maturin-guide.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@messense
Copy link
Member Author

Cause: Check install fail: exit status: 1 
--- Stdout:

--- Stderr:
Traceback (most recent call last):
  File "/home/runner/work/maturin/maturin/test-crates/pyo3-pure/check_installed/check_installed.py", line 5, in <module>
    import pyo3_pure
  File "/usr/share/miniconda/envs/maturin-develop-pyo3-pure-conda/lib/python3.10/site-packages/pyo3_pure/__init__.py", line 1, in <module>
    from .pyo3_pure import *
ImportError: /usr/share/miniconda/envs/maturin-develop-pyo3-pure-conda/lib/python3.10/site-packages/pyo3_pure/pyo3_pure.abi3.so: undefined symbol: PyPyLong_FromSsize_t

thread 'develop_pyo3_pure_conda' panicked at 'Check install fail: exit status: 1 
--- Stdout:

--- Stderr:
Traceback (most recent call last):
  File "/home/runner/work/maturin/maturin/test-crates/pyo3-pure/check_installed/check_installed.py", line 5, in <module>
    import pyo3_pure
  File "/usr/share/miniconda/envs/maturin-develop-pyo3-pure-conda/lib/python3.10/site-packages/pyo3_pure/__init__.py", line 1, in <module>
    from .pyo3_pure import *
ImportError: /usr/share/miniconda/envs/maturin-develop-pyo3-pure-conda/lib/python3.10/site-packages/pyo3_pure/pyo3_pure.abi3.so: undefined symbol: PyPyLong_FromSsize_t

Odd, looks like pyo3 does not rebuild after switching from PyPy to conda.

@messense messense added the CI-no-fail-fast If one job fails, allow the rest to keep testing label Oct 26, 2022
@messense
Copy link
Member Author

Odd, looks like pyo3 does not rebuild after switching from PyPy to conda.

This one is a bit different than #1220 I think.

https://github.com/PyO3/pyo3/blob/8e8b48416940f1ca5899b5c949fd1e24a9211dd8/pyo3-build-config/src/impl_.rs#L1686-L1690

Perhaps the way that cargo:rerun-if-env-changed instructions are set in pyo3-build-config is problematic because they are only conditionally set? But I think PYO3_PYTHON value should have changed which should result in a rebuild, although it seems that didn't happen. I need to reproduce it locally to investigate further.

cc @davidhewitt

@messense
Copy link
Member Author

bors try

bors bot added a commit that referenced this pull request Oct 27, 2022
@bors
Copy link
Contributor

bors bot commented Oct 27, 2022

try

Build failed:

@davidhewitt
Copy link
Member

Hmm. Nothing obvious sticks out to me. I agree with you that it looks like these should be configuring ok. Does the cargo nextest run each build with a separate target directory and/or in sequence (i.e. not in parallel)?

I wonder... is there some nesting of virtualenvs / conda going on? It might be that PyO3 is being forced to ignore e.g. https://github.com/PyO3/pyo3/blob/8e8b48416940f1ca5899b5c949fd1e24a9211dd8/pyo3-build-config/src/impl_.rs#L1669-L1672

... but if maturin is setting PYO3_PYTHON then I don't think this code path should ever run.

@messense
Copy link
Member Author

I wonder... is there some nesting of virtualenvs / conda going on?

FYI, the Python 3.10 issue dates back to #742 (comment)

@messense
Copy link
Member Author

ImportError: dlopen(/Users/runner/work/maturin/maturin/test-crates/venvs/integration-pyo3-mixed-py-subdir-py3.9-pypy/lib/pypy3.9/site-packages/pyo3_mixed_py_subdir/_pyo3_mixed.pypy39-pp73-darwin.so, 6): Symbol not found: _PyModule_GetFilenameObject

Looks like a bug in pyo3?

@messense
Copy link
Member Author

bors try

bors bot added a commit that referenced this pull request Oct 31, 2022
@bors
Copy link
Contributor

bors bot commented Oct 31, 2022

try

Build failed:

@messense
Copy link
Member Author

bors retry

bors bot added a commit that referenced this pull request Oct 31, 2022
@bors
Copy link
Contributor

bors bot commented Oct 31, 2022

try

Build failed:

@messense
Copy link
Member Author

messense commented Nov 1, 2022

bors try

bors bot added a commit that referenced this pull request Nov 1, 2022
@bors
Copy link
Contributor

bors bot commented Nov 1, 2022

try

Build succeeded:

@messense messense marked this pull request as ready for review November 1, 2022 03:41
@messense
Copy link
Member Author

messense commented Nov 1, 2022

bors r+

@bors
Copy link
Contributor

bors bot commented Nov 1, 2022

@bors bors bot merged commit a4fe7cd into PyO3:main Nov 1, 2022
@messense messense deleted the bors branch November 1, 2022 04:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI-no-fail-fast If one job fails, allow the rest to keep testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants