FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

bpo-28334: netrc() now uses expanduser() to find .netrc file by berkerpeksag · Pull Request #4537 · python/cpython · GitHub

Repository navigation

bpo-28334: netrc() now uses expanduser() to find .netrc file - #4537

Merged
berkerpeksag merged 3 commits into
python:masterfrom
berkerpeksag:28334-netrc
Nov 25, 2017
Merged

berkerpeksag merged 3 commits into
python:masterfrom
berkerpeksag:28334-netrc

Conversation

berkerpeksag commented Nov 24, 2017 •
edited
Loading

Copy link
Copy Markdown
Member

Previously, netrc.netrc() was raised an exception if
$HOME is not set.

Authored-By: Dimitri Merejkowsky <dimitri.merejkowsky@tanker.io>

https://bugs.python.org/issue28334

Also add more tests to check that proper exceptions are raised
berkerpeksag reopened this Nov 24, 2017
berkerpeksag changed the title [DO NOT MERGE] bpo-28334: fix netrc not working when $HOME is not set bpo-28334: fix netrc not working when $HOME is not set Nov 25, 2017
berkerpeksag changed the title bpo-28334: fix netrc not working when $HOME is not set bpo-28334: netrc.netrc() now uses os.path.expanduser() to find .netrc Nov 25, 2017
berkerpeksag changed the title bpo-28334: netrc.netrc() now uses os.path.expanduser() to find .netrc bpo-28334: netrc() now uses expanduser() to find .netrc file Nov 25, 2017
berkerpeksag merged commit 8d9bb11 into python:master Nov 25, 2017
berkerpeksag deleted the 28334-netrc branch November 25, 2017 10:37
sloria added a commit to sloria/tinynetrc that referenced this pull request Jan 12, 2018
Avoids an exception when $HOME is not set.
This behavior matches the behavior in Python 3.7
See python/cpython#4537

close #2
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL