This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author Dan McCombs
Recipients Dan McCombs, brett.cannon, python-dev, vinay.sajip, xiang.zhang
Date 2016-04-27.19:42:14
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1461786135.04.0.326282334425.issue26348@psf.upfronthosting.co.za>
In-reply-to
Content
I somehow missed the comment on 2016-04-01, I apologize for being late to look at this.

Xiang is right, this patch should be reverted.

It looks like the issue reported was originally brought on by an activate.fish being used from a Python after the patch in commit 7f913c6ada03 was added, but the running version of Python did not include the patch, so that the placeholder was not being swapped out. In that case, updating the script to include $ and use the variable fixed the problem, but when the patch in 7f913c6ada03 is also included it causes the placeholder to be swapped to a name wrapped in parenthesis preceded by $ at the time the venv is created causing a syntax error in fish.

Since the patch in commit 7f913c6ada03 swaps that place holder and handles doing so in the other activate scripts as well, the patch that was part of this issue (cfc66e37eb8e) is not needed and should be reverted.

Here's the commit adding that placeholder for context:

https://hg.python.org/cpython/rev/7f913c6ada03
History
Date User Action Args
2016-04-27 19:42:15Dan McCombssetrecipients: + Dan McCombs, brett.cannon, vinay.sajip, python-dev, xiang.zhang
2016-04-27 19:42:15Dan McCombssetmessageid: <1461786135.04.0.326282334425.issue26348@psf.upfronthosting.co.za>
2016-04-27 19:42:15Dan McCombslinkissue26348 messages
2016-04-27 19:42:14Dan McCombscreate