Skip to content

Conversation

@vstinner
Copy link
Member

@vstinner vstinner commented Sep 11, 2018

In distutils.command.install, replace "pliant children" (previously,
it was "pliant slaves") with "helpers".

https://bugs.python.org/issue34605

In distutils.command.install, replace "pliant children" (previously,
it was "pliant slaves") with "helpers".
@vstinner
Copy link
Member Author

Copy of @gvanrossum comment:

I Googled a bit and it seems "pliant slave" is an old-fashioned way to refer to slaves, and even more demeaning than just "slave" -- I presume it was meant ironically by the original author. We should really replace the entire construct "pliant slaves" with a more neutral term. I don't think "children" is the right term -- I propose "helpers".

#9101 (comment)

Copy link
Member

@ericvsmith ericvsmith left a comment

Choose a reason for hiding this comment

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

Looks good.

@miss-islington
Copy link
Contributor

@vstinner: Status check is done, and it's a success ✅ .

@miss-islington miss-islington merged commit fa7dfae into python:master Sep 11, 2018
@vstinner vstinner deleted the pliant branch September 11, 2018 23:51
@peter-dolkens

This comment has been minimized.

@brettcannon
Copy link
Member

@dolkensp your insult at Victor about his English skills is uncalled for; you simply could have pointed out that SEO would be affected.

Consider this a warning that insulting people is not tolerated under the Code of Conduct and if it happens again there will be ramifications.

@python python locked as too heated and limited conversation to collaborators Sep 12, 2018
@pitrou
Copy link
Member

pitrou commented Sep 12, 2018

Interesting. As a non-native English speaker myself, I would say that hesitating between "childs" and "children" is a sign that maybe one should wait for comments before merging a language-tweaking PR. Just saying ;-)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants