This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

classification
Title: platform._parse_release_file doesn't close the /etc/lsb-release file, doesn't know about 'Ubuntu'
Type: behavior Stage: patch review
Components: Library (Lib) Versions: Python 3.3, Python 3.4, Python 2.7
process
Status: open Resolution:
Dependencies: Superseder:
Assigned To: Nosy List: christian.heimes, doko, eric.araujo, lemburg, mitya57, pitrou
Priority: normal Keywords: needs review

Created on 2013-04-16 14:49 by doko, last changed 2022-04-11 14:57 by admin.

Files
File name Uploaded Description Edit
pr17761.diff doko, 2013-04-16 14:53
Messages (4)
msg187087 - (view) Author: Matthias Klose (doko) * (Python committer) Date: 2013-04-16 14:49
platform._parse_release_file doesn't close the /etc/lsb-release file, and is missing Ubuntu in _supported_dists.
msg187194 - (view) Author: Antoine Pitrou (pitrou) * (Python committer) Date: 2013-04-17 18:50
Can't you use a "with" statement instead of calling close() explicitly?
msg187388 - (view) Author: Éric Araujo (eric.araujo) * (Python committer) Date: 2013-04-19 22:05
+1 to with.  The patch also includes an unrelated change.
msg201014 - (view) Author: Christian Heimes (christian.heimes) * (Python committer) Date: 2013-10-23 11:06
#17762 suggests /etc/os-release instead of /etc/lsb-release.
History
Date User Action Args
2022-04-11 14:57:44adminsetgithub: 61961
2013-10-23 11:06:48christian.heimessettype: behavior

messages: + msg201014
nosy: + christian.heimes
2013-09-18 07:05:36mitya57setnosy: + mitya57
2013-04-19 22:05:14eric.araujosetnosy: + eric.araujo
messages: + msg187388
2013-04-17 18:50:43pitrousetnosy: + lemburg, pitrou
messages: + msg187194
2013-04-16 14:53:23dokosetkeywords: + needs review, - patch
2013-04-16 14:53:10dokosetfiles: + pr17761.diff
keywords: + patch
2013-04-16 14:49:04dokocreate