Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(4)

Unified Diff: Lib/os.py

Issue 16353: add function to os module for getting path to default shell
Patch Set: Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Please Sign in to add in-line comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | Modules/posixmodule.c » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
--- a/Lib/os.py Sat May 21 14:56:53 2016 +0300
+++ b/Lib/os.py Mon May 23 12:10:08 2016 +0200
@@ -203,6 +203,13 @@
del _add
+if sys.platform == 'win32':
+ default_shell = 'cmd.exe'
Michael.Felt 2016/07/29 14:40:15 Has cmd.exe moved around too much to not specify a
eryksun 2016/07/29 15:03:28 It should be environ['ComSpec'], for which the def
+elif _exists('is_android') and is_android:
+ default_shell = '/system/bin/sh'
+else:
+ default_shell = '/bin/sh'
+
# Python uses fixed values for the SEEK_ constants; they are mapped
# to native constants if necessary in posixmodule.c
# Other possible SEEK values are directly imported from posixmodule.c
« no previous file with comments | « no previous file | Modules/posixmodule.c » ('j') | no next file with comments »

RSS Feeds Recent Issues | This issue
This is Rietveld 894c83f36cb7+