Skip to content

bpo-42163, bpo-42189, bpo-42659: Support uname_tuple._replace (for all but processor) #23010

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 14 commits into from
Dec 31, 2020
Merged

bpo-42163, bpo-42189, bpo-42659: Support uname_tuple._replace (for all but processor) #23010

merged 14 commits into from
Dec 31, 2020

Conversation

jaraco
Copy link
Member

@jaraco jaraco commented Oct 27, 2020

@jaraco jaraco changed the title bpo-42163: Support uname_tuple._replace (for all but processor) bpo-42163, bpo-42189: Support uname_tuple._replace (for all but processor) Oct 29, 2020
@github-actions
Copy link

This PR is stale because it has been open for 30 days with no activity. Remove stale label or comment or this will be closed in 5 days

@github-actions github-actions bot added the stale Stale PR or inactive for long period of time. label Dec 16, 2020
@jaraco jaraco changed the title bpo-42163, bpo-42189: Support uname_tuple._replace (for all but processor) bpo-42163, bpo-42189, bpo-42659: Support uname_tuple._replace (for all but processor) Dec 17, 2020
@jaraco
Copy link
Member Author

jaraco commented Dec 17, 2020

@serhiy-storchaka Would you be willing to take another pass?

@github-actions github-actions bot removed the stale Stale PR or inactive for long period of time. label Dec 18, 2020
@jaraco
Copy link
Member Author

jaraco commented Dec 31, 2020

Rather than let it go stale again, I'm just going to merge.

@jaraco jaraco merged commit a6fd0f4 into python:master Dec 31, 2020
@bedevere-bot
Copy link

@jaraco: Please replace # with GH- in the commit message next time. Thanks!

@miss-islington
Copy link
Contributor

Thanks @jaraco for the PR 🌮🎉.. I'm working now to backport this PR to: 3.9.
🐍🍒⛏🤖

@jaraco jaraco deleted the bugfix/bpo-42163 branch December 31, 2020 19:08
@miss-islington
Copy link
Contributor

Sorry @jaraco, I had trouble checking out the 3.9 backport branch.
Please backport using cherry_picker on command line.
cherry_picker a6fd0f414c0cb4cd5cc20eb2df3340b31c6f7743 3.9

jaraco added a commit that referenced this pull request Jan 16, 2021
…for all but processor) (GH-23010)

* Add test capturing missed expectation with uname_result._replace.

* bpo-42163: Override uname_result._make to allow uname_result._replace to work (for everything but 'processor'.

* Replace hard-coded length with one derived from the definition.

* Add test capturing missed expectation with copy/deepcopy on namedtuple (bpo-42189).

* bpo-42189: Exclude processor parameter when constructing uname_result.

* In _make, rely on __new__ to strip processor.

* Add blurb.

* iter is not necessary here.

* Rely on num_fields in __new__

* Add test for slices on uname

* Add test for copy and pickle.

Co-authored-by: Serhiy Storchaka <[email protected]>

* import pickle

* Fix equality test after pickling.

* Simply rely on __reduce__ for pickling.

Co-authored-by: Serhiy Storchaka <[email protected]>
(cherry picked from commit a6fd0f4)

Co-authored-by: Jason R. Coombs <[email protected]>
@bedevere-bot bedevere-bot removed the needs backport to 3.9 only security fixes label Jan 16, 2021
@bedevere-bot
Copy link

GH-24232 is a backport of this pull request to the 3.9 branch.

jaraco added a commit that referenced this pull request Jan 16, 2021
…for all but processor) (GH-23010) (#24232)

* Add test capturing missed expectation with uname_result._replace.

* bpo-42163: Override uname_result._make to allow uname_result._replace to work (for everything but 'processor'.

* Replace hard-coded length with one derived from the definition.

* Add test capturing missed expectation with copy/deepcopy on namedtuple (bpo-42189).

* bpo-42189: Exclude processor parameter when constructing uname_result.

* In _make, rely on __new__ to strip processor.

* Add blurb.

* iter is not necessary here.

* Rely on num_fields in __new__

* Add test for slices on uname

* Add test for copy and pickle.

Co-authored-by: Serhiy Storchaka <[email protected]>

* import pickle

* Fix equality test after pickling.

* Simply rely on __reduce__ for pickling.

Co-authored-by: Serhiy Storchaka <[email protected]>
(cherry picked from commit a6fd0f4)

Co-authored-by: Jason R. Coombs <[email protected]>
adorilson pushed a commit to adorilson/cpython that referenced this pull request Mar 13, 2021
…l but processor) (python#23010)

* Add test capturing missed expectation with uname_result._replace.

* bpo-42163: Override uname_result._make to allow uname_result._replace to work (for everything but 'processor'.

* Replace hard-coded length with one derived from the definition.

* Add test capturing missed expectation with copy/deepcopy on namedtuple (bpo-42189).

* bpo-42189: Exclude processor parameter when constructing uname_result.

* In _make, rely on __new__ to strip processor.

* Add blurb.

* iter is not necessary here.

* Rely on num_fields in __new__

* Add test for slices on uname

* Add test for copy and pickle.

Co-authored-by: Serhiy Storchaka <[email protected]>

* import pickle

* Fix equality test after pickling.

* Simply rely on __reduce__ for pickling.

Co-authored-by: Serhiy Storchaka <[email protected]>
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