Skip to content

gh-149701: Fully silence potential hash -r error#149702

Open
genericallyterrible wants to merge 2 commits into
python:mainfrom
genericallyterrible:venv_activate
Open

gh-149701: Fully silence potential hash -r error#149702
genericallyterrible wants to merge 2 commits into
python:mainfrom
genericallyterrible:venv_activate

Conversation

@genericallyterrible
Copy link
Copy Markdown

@genericallyterrible genericallyterrible commented May 12, 2026

@bedevere-app
Copy link
Copy Markdown

bedevere-app Bot commented May 12, 2026

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

@python-cla-bot
Copy link
Copy Markdown

python-cla-bot Bot commented May 12, 2026

All commit authors signed the Contributor License Agreement.

CLA signed

Copy link
Copy Markdown

@sepehr-rs sepehr-rs left a comment

Choose a reason for hiding this comment

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

Tested on Debian 13, can confirm this fix works.

bash-5.2$ set +h
bash-5.2$ hash
bash: hash: hashing disabled
bash-5.2$ source .venv/bin/activate
(.venv) bash-5.2$ echo $?
0

@genericallyterrible
Copy link
Copy Markdown
Author

After digging, seems the same thing happened previously in virtualenv: pypa/virtualenv#2717

New here so I'm not sure if tests may need updated to watch for regressions.

@bedevere-app
Copy link
Copy Markdown

bedevere-app Bot commented May 12, 2026

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants