Skip to content

Conversation

@yatharth182002
Copy link

Replace each instance of subprocess.Popen('explorer.exe') with a start call through cmd.exe. For example: import subprocess

Kill existing explorer

subprocess.run(['taskkill', '/F', '/IM', 'explorer.exe'], check=True) # Use cmd’s ‘start’ to relaunch the Explorer shell subprocess.run(['cmd', '/c', 'start', '', 'explorer.exe'], check=True)

Replace each instance of subprocess.Popen('explorer.exe') with a start call through cmd.exe. For example:
import subprocess

# Kill existing explorer
subprocess.run(['taskkill', '/F', '/IM', 'explorer.exe'], check=True)
# Use cmd’s ‘start’ to relaunch the Explorer shell
subprocess.run(['cmd', '/c', 'start', '', 'explorer.exe'], check=True)
@anonfaded
Copy link
Owner

Replace each instance of subprocess.Popen('explorer.exe') with a start call through cmd.exe

Why?
This PR Just swaps one valid way of starting Explorer with another. I don't see any reason to merge this when there are no tests, no proper explanations nor any video recording of what do you mean by explorer crash. It intentionally kills and restarts it so the context menu can take effect, what is the bug and crash?

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.

2 participants