GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — bpo-22257: Private C-API for main interpreter initialization (PEP 432). by ericsnowcurrently · Pull Request #1729 · python/cpython

Conversation

@ericsnowcurrently

@ericsnowcurrently ericsnowcurrently commented May 22, 2017 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member

(Patch by Nick Coghlan)

(based on #1772)

This patch introduces the bulk of the remaining changes specified by PEP 432. These are introduced here as "private" C-API while the PEP is discussed. Regardless of the status of the PEP, the changes here are an improvement to interpreter startup. As such we are making the changes now, but keeping the C-API private.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@the-knights-who-say-ni the-knights-who-say-ni added the CLA signed label May 22, 2017
@mention-bot

mention-bot commented May 22, 2017

Copy link
Copy Markdown

@ericsnowcurrently, thanks for your PR! By analyzing the history of the files in this pull request, we identified @serhiy-storchaka, @methane and @loewis to be potential reviewers.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@ericsnowcurrently

ericsnowcurrently commented May 22, 2017

Copy link
Copy Markdown
Member Author

@zooba
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@ericsnowcurrently
ericsnowcurrently force-pushed the pep432 branch 5 times, most recently from 6da8498 to 90c5a8f Compare May 23, 2017 04:43
@ericsnowcurrently ericsnowcurrently changed the title bpo-22257: Private C-API for PEP 432. bpo-22257: Private C-API for main interpreter initialization (PEP 432). May 23, 2017
@ericsnowcurrently
ericsnowcurrently force-pushed the pep432 branch from 90c5a8f to faf2f8e Compare May 23, 2017 22:56
zooba
zooba reviewed May 24, 2017
View reviewed changes
Comment thread Python/pythonrun.c Outdated

@zooba zooba May 24, 2017

Copy link
Copy Markdown
Member

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

Delete this

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
zooba
zooba approved these changes May 24, 2017
View reviewed changes
ncoghlan and others added 2 commits May 23, 2017 21:48
@ncoghlan @ericsnowcurrently
Add _PyMainInterpreterConfig and _Py_InitializeMainInterpreter().
645c733
@ericsnowcurrently
Drop an outdated TODO.
bcd1b52
@ericsnowcurrently
ericsnowcurrently force-pushed the pep432 branch from faf2f8e to bcd1b52 Compare May 24, 2017 04:50
@ericsnowcurrently
ericsnowcurrently merged commit c7ec998 into python:master May 24, 2017
@ericsnowcurrently
ericsnowcurrently deleted the pep432 branch May 24, 2017 06:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@zooba zooba zooba approved these changes

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

5 participants

@ericsnowcurrently @mention-bot @zooba @the-knights-who-say-ni @ncoghlan
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.