-
-
Notifications
You must be signed in to change notification settings - Fork 4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Allow users to specify Python executable path #19644
Conversation
src/gui/advancedsettings.cpp
Outdated
// Python executable path | ||
m_pythonExecutablePath.setPlaceholderText(tr("(Auto detect if empty)")); | ||
m_pythonExecutablePath.setText(pref->getPythonExecutablePath().toString()); | ||
addRow(PYTHON_EXECUTABLE_PATH, tr("Python executable path (requires restart)"), &m_pythonExecutablePath); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The requires restart
isn't always. There are some cases that simply reloading the search tab will have immediate effect. But in order to avoid usage confusion, I decided to add this wording.
And pardon, I currently can't afford the time to redesign and make the experience better. Hopefully the current form will be enough for those who demands it.
Will this alternate python path be displayed in the About info tab ? (to help troubleshoot/triage problems) |
No but the log will record some status. |
Quick test....ran in |
Seems all is fine to me. Python is found eventually:
|
To me too.....only posted a screenshot for other to see log messages & where in settings it could be found... |
Too busy. I'll try to review it tomorrow. |
Should close #18557 & probably others too. |
This setting has indeed no effect on macOS Sequoia 15.0.1 (24A348). The default Python version is 3.12.6, causing qBittorrent Search to break. |
Closes #19195.