Skip to content

Conversation

@vstinner
Copy link
Member

@vstinner vstinner commented Sep 24, 2018

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