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

bpo-20104: Remove posix_spawn from 3.7 by pablogsal · Pull Request #6794 · python/cpython · GitHub

  1. python
  2. cpython

Repository navigation

bpo-20104: Remove posix_spawn from 3.7 - #6794

Merged
gpshead merged 3 commits into
python:3.7from
pablogsal:remove_posix_spawn
May 14, 2018
ConversationCommits (3)ChecksFiles changed
Merged

bpo-20104: Remove posix_spawn from 3.7#6794
gpshead merged 3 commits into
python:3.7from
pablogsal:remove_posix_spawn

Conversation

pablogsal commented May 14, 2018 •
edited by bedevere-bot
Loading

Copy link
Copy Markdown
Member

pablogsal changed the title bpo-20104: Remove posix_spawn in 3.7 bpo-20104: Remove posix_spawn from 3.7 May 14, 2018
pablogsal force-pushed the remove_posix_spawn branch 2 times, most recently from 3284174 to c200683 Compare May 14, 2018 15:10

pablogsal commented May 14, 2018

Copy link
Copy Markdown
Member Author

CC: @gpshead

pablogsal force-pushed the remove_posix_spawn branch from c200683 to a65ef05 Compare May 14, 2018 16:01
gpshead reviewed May 14, 2018
View reviewed changes

gpshead 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.

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

this looks straight forward.

I suggest dropping the changes to aclocal.m4, configure.ac, configure, pyconfig.h.in that remove the check for the API. It is fine for us to leave the autoconf script in 3.7 testing for the posix_spawn function.

gpshead commented May 14, 2018 •
edited
Loading

Copy link
Copy Markdown
Member

@ned-deily - what would you like to see for a Misc/NEWS.d entry for this. os.posix_spawn appeared in some betas but we're removing it before rc1. document it as being removed again? or just remove all prior mentions of adding it?

gpshead self-assigned this May 14, 2018

gpshead commented May 14, 2018

Copy link
Copy Markdown
Member

err, wrong autocompleted. ned. sorry. :) @ned-deily :)

gpshead requested a review from ned-deily May 14, 2018 16:23
gpshead added 2 commits May 14, 2018 11:35
Add a news entry.
6a4a373 
and reference the removal from the old news entry where it was added.
Undo the autoconf related changes.
5700952 
We can keep the configure test for posix_spawn, we'll need it in the
future and having it in pyconfig.h will be convenient in case any
3.7 extension module wants to use it.

gpshead commented May 14, 2018 •
edited
Loading

Copy link
Copy Markdown
Member

okay, i've undone the configure changes and done the news entries. just waiting for the various CI runs to complete (overloaded by the sprints) before merging.

gpshead removed the request for review from ned-deily May 14, 2018 18:48
gpshead merged commit 8e633a4 into python:3.7 May 14, 2018
pablogsal deleted the remove_posix_spawn branch May 14, 2018 22:06
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
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

gpshead gpshead left review comments
serhiy-storchaka serhiy-storchaka approved these changes

Assignees

gpshead

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

Back | FazBrowse Home | New Git URL