FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

gh-133537: only use console when available by maxbachmann · Pull Request #133538 · python/cpython · GitHub

/ cpython Public

gh-133537: only use console when available - #133538

Merged
zooba merged 2 commits into
python:mainfrom
maxbachmann:patch-2
May 7, 2025
Merged

gh-133537: only use console when available#133538
zooba merged 2 commits into
python:mainfrom
maxbachmann:patch-2

Conversation

maxbachmann commented May 6, 2025
edited by bedevere-app Bot
Loading

Copy link
Copy Markdown
Contributor

Alternatively to always returning false the function could only be available when the partition supports console io. Similar to the functions in winconsoleio which are only available when the partition supports console io.

sharktide left a comment
edited
Loading

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Please explain your changes and how they help in a more understandable way so the review process can go a lot faster :) TiA

maxbachmann commented May 6, 2025
edited
Loading

Copy link
Copy Markdown
Contributor Author

GetConsoleMode is only available when HAVE_WINDOWS_CONSOLE_IO is defined. Using it without the guard fails to compile on platforms like the xbox where console io isn't available.

I wasn't sure whether this should get a news entry. If it should I can add one.

sharktide left a comment
edited
Loading

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

I wasn't sure wheter this should get a news entry. If it should I can add one

In this case it doesn't really matter, since XBox and similar partitions aren't commonly used by CPython developers.

@ZeroIntensity what do you think

Copy link
Copy Markdown
Member

I have absolutely no idea if we support compiling on XBox (if we do, that's sort of hilarious), but we do accept changes for unsupported platforms, which do need blurb entries. I'm not enough of an expert here to deal with this any further.

@chris-eibl, I heard you know a thing or two about Windows. Would you mind looking at this?

Copy link
Copy Markdown
Member

@maxbachmann already improved building on xbox in the past - #102256 was quite a big task.

This looks good to me, just needs a blurb entry.

Pinging @zooba for final steps.

chris-eibl added OS-windows interpreter-core (Objects, Python, Grammar, and Parser dirs) build The build process and cross-build labels May 7, 2025

zooba commented May 7, 2025

Copy link
Copy Markdown
Member

Going to have to wait until beta 1 gets released, but yeah, this is fine.

And we totally support building on Xbox, plenty of games have embedded Python in the past and we want that to be able to continue. We don't have a ready release though, but Max has done a ton of work in the past to make it work nicely. Xbox OS is basically just a modified version of Windows1, so everything works the same, but there are some differences in what APIs are available so that it's easier to trust any apps.

Footnotes

  1. Very vague description, but I'm not sure how much I can share publicly about how it's put together.

sharktide left a comment
edited
Loading

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

This PR seems to be good, but we will have to wait until Beta 1 gets released as zooba said

zooba merged commit 1460cce into python:main May 7, 2025
Pranjal095 pushed a commit to Pranjal095/cpython that referenced this pull request Jul 12, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build The build process and cross-build interpreter-core (Objects, Python, Grammar, and Parser dirs) OS-windows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants


Back | FazBrowse Home | New Git URL