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: Move _findvs into its own extension module
Type: performance Stage: resolved
Components: Distutils, Windows Versions: Python 3.8, Python 3.7
process
Status: closed Resolution: fixed
Dependencies: Superseder:
Assigned To: steve.dower Nosy List: dstufft, eric.araujo, paul.moore, steve.dower, tim.golden, zach.ware
Priority: normal Keywords: patch

Created on 2018-01-17 22:09 by steve.dower, last changed 2022-04-11 14:58 by admin. This issue is now closed.

Pull Requests
URL Status Linked Edit
PR 5227 merged steve.dower, 2018-01-18 04:02
PR 5228 merged steve.dower, 2018-01-18 04:32
Messages (4)
msg310209 - (view) Author: Steve Dower (steve.dower) * (Python committer) Date: 2018-01-17 22:09
The _findvs module I added for distutils has added load-time dependencies on the ole32 and oleaut32 DLLs.

To reduce startup time, we should move _findvs into its own .pyd (unless I hear a better suggestion, I like "distutils._findvs.pyd").
msg310211 - (view) Author: Steve Dower (steve.dower) * (Python committer) Date: 2018-01-17 22:11
Actually, I'd prefer not dropping it into the Lib directory, so probably DLLs/_distutils_findvs.pyd is the way to go.
msg310258 - (view) Author: Steve Dower (steve.dower) * (Python committer) Date: 2018-01-18 22:09
New changeset 2507e29a9e4e9dcac6eab46546bd3d34a69342ba by Steve Dower in branch 'master':
bpo-32588: Move _findvs into its own module and add missing _queue module to installer (#5227)
https://github.com/python/cpython/commit/2507e29a9e4e9dcac6eab46546bd3d34a69342ba
msg310259 - (view) Author: Steve Dower (steve.dower) * (Python committer) Date: 2018-01-18 22:09
New changeset ccf7f05c5d3fdea89d857e775d2c6371f3e15b4a by Steve Dower in branch '3.6':
[3.6] bpo-32588 Move _distutils_findvs into its own module (GH-5227) (#5228)
https://github.com/python/cpython/commit/ccf7f05c5d3fdea89d857e775d2c6371f3e15b4a
History
Date User Action Args
2022-04-11 14:58:56adminsetgithub: 76769
2018-01-18 22:10:24steve.dowersetstatus: open -> closed
resolution: fixed
stage: patch review -> resolved
2018-01-18 22:09:51steve.dowersetmessages: + msg310259
2018-01-18 22:09:42steve.dowersetmessages: + msg310258
2018-01-18 04:32:16steve.dowersetpull_requests: + pull_request5075
2018-01-18 04:02:53steve.dowersetkeywords: + patch
stage: needs patch -> patch review
pull_requests: + pull_request5074
2018-01-18 03:33:41steve.dowersetassignee: steve.dower
2018-01-17 22:11:12steve.dowersetmessages: + msg310211
2018-01-17 22:09:56steve.dowercreate