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.

Title: PC\bdist_wininst\install.c: Missing call to fclose()
Type: Stage: resolved
Components: Windows Versions:
Status: closed Resolution: wont fix
Dependencies: Superseder:
Assigned To: Nosy List: Josh Snider, aatishnn, maddin200, martin.panter, paul.moore, steve.dower, tim.golden, vstinner, zach.ware
Priority: normal Keywords: easy, patch

Created on 2016-03-14 10:01 by maddin200, last changed 2022-04-11 14:58 by admin. This issue is now closed.

File name Uploaded Description Edit
issue26554.patch aatishnn, 2016-03-19 04:00 For 3.6 review
Messages (4)
msg261737 - (view) Author: (maddin200) Date: 2016-03-14 10:01
cpython\PC\bdist_wininst\install.c line 2542
missing fclose(logfile);
msg263597 - (view) Author: Josh Snider (Josh Snider) * Date: 2016-04-17 03:34
I suggested some changes to your patch. Your patch should also add your name to the Misc/ACKS file as I don't currently see an Aatish listed there.
msg263601 - (view) Author: Martin Panter (martin.panter) * (Python committer) Date: 2016-04-17 08:28
Adding aatishnn to nosy in case they aren’t aware of your review.
msg384821 - (view) Author: STINNER Victor (vstinner) * (Python committer) Date: 2021-01-11 12:51
The distutils bdist_wininst command has been removed in Python 3.10: see bpo-42802.
Date User Action Args
2022-04-11 14:58:28adminsetgithub: 70741
2021-01-11 12:51:35vstinnersetstatus: open -> closed

nosy: + vstinner
messages: + msg384821

resolution: wont fix
stage: resolved
2016-04-17 08:28:19martin.pantersetnosy: + aatishnn, martin.panter
messages: + msg263601
2016-04-17 03:34:44Josh Snidersetnosy: + Josh Snider
messages: + msg263597
2016-03-19 04:00:48aatishnnsetfiles: + issue26554.patch
keywords: + patch
2016-03-17 16:39:54vstinnersettitle: Missing fclose -> PC\bdist_wininst\install.c: Missing call to fclose()
2016-03-17 16:39:39vstinnersetkeywords: + easy
2016-03-14 10:01:41maddin200create