Author chris.jerdonek
Recipients asvetlov, chris.jerdonek, christian.heimes, ezio.melotti, gregory.p.smith, lyapun, neologix, pitrou, r.david.murray, tim.golden
Date 2012-11-03.14:13:19
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1351952000.21.0.0969914990744.issue16353@psf.upfronthosting.co.za>
In-reply-to
Content
+++ b/Lib/shutil.py	Sat Nov 03 13:32:05 2012 +0200
+
+def get_default_shell_path():

Why is the patch putting the function in the shutil module?  The function should go in the os module as the title and comments of this issue state.  shutil seems misplaced to me.

For example, the description of shutil in the docs is, "The shutil module offers a number of high-level operations on files and collections of files.  In particular, functions are provided which support file copying and removal."  In contrast, the description of the os module is, "This module provides a portable way of using operating system dependent functionality."  The default shell is operating system dependent functionality.
History
Date User Action Args
2012-11-03 14:13:20chris.jerdoneksetrecipients: + chris.jerdonek, gregory.p.smith, pitrou, christian.heimes, tim.golden, ezio.melotti, r.david.murray, asvetlov, neologix, lyapun
2012-11-03 14:13:20chris.jerdoneksetmessageid: <1351952000.21.0.0969914990744.issue16353@psf.upfronthosting.co.za>
2012-11-03 14:13:20chris.jerdoneklinkissue16353 messages
2012-11-03 14:13:19chris.jerdonekcreate