Skip to content

[3.7] bpo-34783: Fix Py_Main()#9526

Merged
vstinner merged 1 commit into
python:3.7from
vstinner:pymain_local_config
Sep 24, 2018
Merged

[3.7] bpo-34783: Fix Py_Main()#9526
vstinner merged 1 commit into
python:3.7from
vstinner:pymain_local_config

Conversation

@vstinner

@vstinner vstinner commented Sep 24, 2018

Copy link
Copy Markdown
Member

Fix a crash with musl libc (on Alpine Linux) when the script filename
specified on the command line doesn't exist. pymain_open_filename()
now gets the current core configuration from the interpreter state.

Modify the code to make it closer to the master branch:

  • Rename _Py_CommandLineDetails to _PyCmdline
  • Remove _PyMain.config: replaced with a local variable
    'local_config' in pymain_init()
  • Reorganize pymain_main(): move code using the "local config"
    into pymain_init()
  • As soon as possible, switch from the local config to the core
    configuration attached to the interpreter.

https://bugs.python.org/issue34783

Fix a crash with musl libc (on Alpine Linux) when the script filename
specified on the command line doesn't exist. pymain_open_filename()
now gets the current core configuration from the interpreter state.

Modify the code to make it closer to the master branch:

* Rename _Py_CommandLineDetails to _PyCmdline
* Remove _PyMain.config: replaced with a local variable
  'local_config' in pymain_init()
* Reorganize pymain_main(): move code using the "local config"
  into pymain_init()
* As soon as possible, switch from the local config to the core
  configuration attached to the interpreter.
@vstinner vstinner merged commit ddc163d into python:3.7 Sep 24, 2018
@vstinner vstinner deleted the pymain_local_config branch September 24, 2018 12:03
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.

3 participants