[Patches] [ python-Patches-1574252 ] Add %var% support to ntpath.expandvars

[SourceForge.net]

Patches item #1574252, was opened at 2006-10-10 04:45
Message generated for change (Comment added) made by gbrandl
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1574252&group_id=5470

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Library (Lib)
Group: None
>Status: Closed
>Resolution: Out of Date
Priority: 5
Private: No
Submitted By: Chip Norkus (doubleyewdee)
Assigned to: Nobody/Anonymous (nobody)
Summary: Add %var% support to ntpath.expandvars

Initial Comment:
NT variables are passed around as %var% instead of 
$var or ${var}.  Providing support for this gives a 
seamless user experience for NT users using Python.

E.g. os.path.expandvars('%TEMP%') now does what is 
expected on NT.

I updated the docstring, could not find any tests or 
documentation references in svn for this portion of 
code.

----------------------------------------------------------------------

>Comment By: Georg Brandl (gbrandl)
Date: 2007-03-13 08:40

Message:
Logged In: YES 
user_id=849994
Originator: NO

Closing in favor of #957650.

----------------------------------------------------------------------

Comment By: Josiah Carlson (josiahcarlson)
Date: 2006-10-22 21:28

Message:
Logged In: YES 
user_id=341410

I've added a new patch to http://python.org/sf/957650 that
adds both the expanduser and expandvars case.  The
expandvars case is slightly different from what the OP
submitted, if only in that it won't mask ValueErrors that
don't come from the str.index() call.

----------------------------------------------------------------------

Comment By: Josiah Carlson (josiahcarlson)
Date: 2006-10-22 17:49

Message:
Logged In: YES 
user_id=341410

I believe that there are two changes that are desireable;
the functionality offered in this patch (expansion of %VAR%
environment variables), and ~user\subpath expansion. 
Recursive re-application of variable expansion shouldn't
occur.  I can merge the two patches that adds the
functionality to ntpath.expanduser and ntpath.expandvars .

----------------------------------------------------------------------

Comment By: Martin v. Löwis (loewis)
Date: 2006-10-22 14:36

Message:
Logged In: YES 
user_id=21627

josiahcarlson, can you propose a strategy to resolve this all?


----------------------------------------------------------------------

Comment By: Josiah Carlson (josiahcarlson)
Date: 2006-10-16 22:47

Message:
Logged In: YES 
user_id=341410

See also: http://python.org/sf/796219 and
http://python.org/sf/957650

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1574252&group_id=5470