shell-history not working with Python virtual environments on Debian Bookworm

[juls]

(At least) for zsh and bash, this line breaks with Pythons virtual environments on Debian Bookworm.

To reproduce,

• create a new Dev Container at .devcontainer/devcontainer.json with

{
	"name": "Python 3",
	"image": "mcr.microsoft.com/devcontainers/python:1-3.11-bookworm",
	"features": {
		"ghcr.io/stuartleeks/dev-container-features/shell-history:0": {}
	}
}

• open the Dev Container and Create a Python environment as described here
• Open a new shell in the terminal window and you'll see

source /workspaces/python-test/.venv/bin/activate
source /workspaces/python-test/.venv/bin/activate
vscode ➜ /workspaces/python-test $

with no activated environment.

• if you switch to :1-3.11-bullseye as the image source, it works (after a Python:Select Interpreter from the Command Palette):

source /workspaces/python-test/.venv/bin/activate
vscode ➜ /workspaces/python-test $ source /workspaces/python-test/.venv/bin/activate
(.venv) vscode ➜ /workspaces/python-test $ 

If you switch back to 1-3.11-bookworm it won't work again unless you delete the line mentioned above.

[juls]

@stuartleeks I saw that you also addressed this issue in 61fb446 but then reverted the fix; by changing owner of /dc/shellhistory during installation (while root) there should be no need to chown at login anymore.

I've only tested this for bash and zsh.