Skip to content

[3.7] bpo-33358: Fix test_embed.test_pre_initialization_sys_options (GH-6612)#6618

Merged
miss-islington merged 1 commit into
python:3.7from
miss-islington:backport-4114846-3.7
Apr 27, 2018
Merged

[3.7] bpo-33358: Fix test_embed.test_pre_initialization_sys_options (GH-6612)#6618
miss-islington merged 1 commit into
python:3.7from
miss-islington:backport-4114846-3.7

Conversation

@miss-islington

@miss-islington miss-islington commented Apr 27, 2018

Copy link
Copy Markdown
Contributor

Fix test_embed.test_pre_initialization_sys_options() when building with --enable-shared
(cherry picked from commit 4114846)

Co-authored-by: Pablo Galindo Pablogsal@gmail.com

https://bugs.python.org/issue33358

…GH-6612)

Fix test_embed.test_pre_initialization_sys_options() when building with --enable-shared
(cherry picked from commit 4114846)

Co-authored-by: Pablo Galindo <Pablogsal@gmail.com>

@vstinner vstinner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, good bot.

@vstinner

Copy link
Copy Markdown
Member

I tested manually Python 3.7 with --enabled-shared: test_embed fails without this change, and pass with this change.

@vstinner

Copy link
Copy Markdown
Member

cc @pablogsal

@miss-islington

Copy link
Copy Markdown
Contributor Author

@pablogsal and @vstinner: Backport status check is done, and it's a success ✅ .

@miss-islington miss-islington merged commit dd3ede7 into python:3.7 Apr 27, 2018
@miss-islington

Copy link
Copy Markdown
Contributor Author

Thanks!

@miss-islington miss-islington deleted the backport-4114846-3.7 branch April 27, 2018 12:41
@vstinner

Copy link
Copy Markdown
Member

@pablogsal and @vstinner: Backport status check is done, and it's a success white_check_mark .

Oh wow, that was fast. Now I really saw that bot doing an actual merge. Impressive work @Mariatta!

@vstinner

Copy link
Copy Markdown
Member

Thanks again, good bot!

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.

5 participants