Skip to content

Python: Add support for Python 3.12#35

Closed
steinsgateted wants to merge 8 commits into
apache:mainfrom
steinsgateted:python-3.12
Closed

Python: Add support for Python 3.12#35
steinsgateted wants to merge 8 commits into
apache:mainfrom
steinsgateted:python-3.12

Conversation

@steinsgateted

Copy link
Copy Markdown

@Fokko

Fokko commented Oct 2, 2023

Copy link
Copy Markdown
Contributor

Thanks for opening this PR. 3.12 has been released today: https://www.python.org/downloads/release/python-3120/

@Fokko

Fokko commented Oct 13, 2023

Copy link
Copy Markdown
Contributor

@steinsgateted can you pull in the main branch? It looks like that 3.12 is available: https://github.com/actions/python-versions/releases

@Fokko Fokko added this to the PyIceberg 0.6.0 release milestone Oct 13, 2023
@steinsgateted

Copy link
Copy Markdown
Author

@Fokko Thank you.
Encountered this problem:

Note: This error originates from the build backend, and is likely not a problem with poetry but with aiohttp (3.8.5) not supporting PEP 517 builds. You can verify this by running 'pip wheel --use-pep517 "aiohttp (==3.8.5)"'.

@jayceslesar

Copy link
Copy Markdown
Contributor

@steinsgateted looks like there are no 3.12 wheels yet see the discussion on aio-libs/aiohttp#7639

@steinsgateted

Copy link
Copy Markdown
Author

@jayceslesar Thank you for the information

@Fokko Fokko mentioned this pull request Nov 22, 2023
@Fokko

Fokko commented Nov 22, 2023

Copy link
Copy Markdown
Contributor

Thanks for picking this up again, it looks like it is now blocked on Poetry: #158

@steinsgateted

steinsgateted commented Nov 22, 2023

Copy link
Copy Markdown
Author

@Fokko @jayceslesar Thank you

@steinsgateted looks like there are no 3.12 wheels yet see the discussion on aio-libs/aiohttp#7639

I found that 3.9.0b0 supports python3.12.
Do we change 3.8.6 to 3.9?

@MehulBatra

Copy link
Copy Markdown
Contributor

Looking into this

@Fokko

Fokko commented Mar 13, 2024

Copy link
Copy Markdown
Contributor

@steinsgateted can you rebase the PR?

@kevinjqliu

Copy link
Copy Markdown
Contributor

Closing this in favor of #1068

@kevinjqliu kevinjqliu closed this Sep 9, 2024
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.

6 participants