Dimitri Merejkowsky added the comment:
> the inevitable confusion when multiple PATH updates do not play well together
> (which they don't).
Oh I see. Did not think about that. I can see a scenario where you install 3.9,
PATH is set, you install 3.10, the PATH update fails for some
Dimitri Merejkowsky added the comment:
I know this is two years later so maybe this is not the best way to get
answers, but Steve Dower said:
> We don't add Python to PATH by default (because it's bad), and people
> couldn't find the option
Could someone clarify this for me? Beca
Changes by Dimitri Merejkowsky <d.me...@gmail.com>:
--
pull_requests: +1227
___
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue3041>
___
New submission from Dimitri Merejkowsky:
If you have:
subprocess.run(["ls", "--help"], shell=True)
you'll see that the command run is actually just "ls", not "ls --help"
--
components: Library (Lib)
messages: 291473
nosy: Dimitri Merejkowsky
New submission from Dimitri Merejkowsky:
Motivation for the patch came from a tweet[1] of David Beazley:
def SomeError(Exception):
pass
raise SomeError('blah')
(Note the `def` keyword instead of `class`):
If you run that, you get:
> TypeError: exceptions must derive from BaseExcept
Dimitri Merejkowsky added the comment:
During review SilentGhost suggested that maybe a test was not essential.
In any case, I think patching documentation about the new behavior won't hurt.
Do you want me to do that?
--
___
Python tracker <
Dimitri Merejkowsky added the comment:
> if it make sense to run this on Windows
I found this issue while running cross-platform code. I needed to store some
credentials, did not mind having them in plain-text and I thought .netrc was a
good place for this. (did not need to re-inv
Dimitri Merejkowsky added the comment:
> However, since we are changing the behavior of the netrc() class, I'm not
> sure this can be considered as a bug fix.
I was not sure either. The patch does change behavior in subtle ways...
I've added a new patch, and sent the CLA form.
New submission from Dimitri Merejkowsky:
If $HOME is not set, netrc will raise an exception.
Attached patch fixes the problem by using `os.path.expanduser` instead
--
components: Library (Lib)
files: netrc-use-expanduser.patch
keywords: patch
messages: 277824
nosy: Dimitri Merejkowsky
