Skip to content

bpo-32155: Bugfixes found by flake8 F841 warnings#4608

Merged
vstinner merged 1 commit into
python:masterfrom
vstinner:F841_bugfixes
Nov 28, 2017
Merged

bpo-32155: Bugfixes found by flake8 F841 warnings#4608
vstinner merged 1 commit into
python:masterfrom
vstinner:F841_bugfixes

Conversation

@vstinner
Copy link
Copy Markdown
Member

@vstinner vstinner commented Nov 28, 2017

  • distutils.config: Use the PyPIRCCommand.realm attribute if set
  • turtledemo: wait until macOS osascript command completes to not
    create a zombie process
  • Tools/scripts/treesync.py: declare 'default_answer' and
    'create_files' as globals to modify them with the command line
    arguments. Previously, -y, -n, -f and -a options had no effect.

flake8 warning: "F841 local variable 'p' is assigned to but never
used".

https://bugs.python.org/issue32155

* distutils.config: Use the PyPIRCCommand.realm attribute if set
* turtledemo: wait until macOS osascript command completes to not
  create a zombie process
* Tools/scripts/treesync.py: declare 'default_answer' and
  'create_files' as globals to modify them with the command line
  arguments. Previously, -y, -n, -f and -a options had no effect.

flake8 warning: "F841 local variable 'p' is assigned to but never
used".
@vstinner vstinner merged commit 696b501 into python:master Nov 28, 2017
@miss-islington
Copy link
Copy Markdown
Contributor

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

@vstinner vstinner deleted the F841_bugfixes branch November 28, 2017 14:30
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Nov 28, 2017
* distutils.config: Use the PyPIRCCommand.realm attribute if set
* turtledemo: wait until macOS osascript command completes to not
  create a zombie process
* Tools/scripts/treesync.py: declare 'default_answer' and
  'create_files' as globals to modify them with the command line
  arguments. Previously, -y, -n, -f and -a options had no effect.

flake8 warning: "F841 local variable 'p' is assigned to but never
used".
(cherry picked from commit 696b501)
@bedevere-bot
Copy link
Copy Markdown

GH-4610 is a backport of this pull request to the 3.6 branch.

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.

4 participants