valhallasw created this task. valhallasw added a subscriber: valhallasw. valhallasw added a project: pywikibot-core. TASK DESCRIPTION e.g. ``` editor=r'C:/Program Files (x86)/Notepad++/notepad++.exe' ``` would run ``` os.system(r'C:/Program Files (x86)/Notepad++/notepad++.exe -n0 tempfilename') ``` which is interpreted as 'run `c:/program` with arguments `Files`, `(x86)`, ...'. We should probably switch to subprocess.Popen, and pass the extra parameters as a tuple, i.e., as ``` subprocess.Popen((r'C:/Program Files (x86)/Notepad++/notepad++.exe', '-n0', 'tempfilename')) ``` which should take care of all escaping. Workarounds: - Double-quote, i.e. `editor=r'"C:/Program Files (x86)/Notepad++/notepad++.exe"'` - use 8.3 notation for parts with spaces, i.e. `editor=r'C:/Progra~2/Notepad++/notepad++.exe'` TASK DETAIL https://phabricator.wikimedia.org/T86481 REPLY HANDLER ACTIONS Reply to comment or attach files, or !close, !claim, !unsubscribe or !assign <username>. EMAIL PREFERENCES https://phabricator.wikimedia.org/settings/panel/emailpreferences/ To: valhallasw Cc: Aklapper, valhallasw, jayvdb, pywikipedia-bugs