[New-bugs-announce] [issue18517] "xxlimited" extension declared incorrectly in setup.py

[Ned Deily]

New submission from Ned Deily:

In the top-level setup.py, the code to define the Extension instance for the "xxlimited" extension is currently incorrectly located near the end of detect_tkinter() rather than in its parent detect_modules(). This has the effect of skipping the build of "xxlimited" on OS X since detect_tkinter() shortcuts to detect_tkinter_darwin() there.  The attached patch corrects that.

However, it does raise the question of why "xxlimited" is built at all for normal installable builds, since "xxlimited" is a dummy template module.  The definition for the somewhat similar "xx" is commented out by default.  Perhaps "xxlimited" should be as well.  Or perhaps both should only be built for --with-pydebug configs.  Opinions?

----------
components: Build
messages: 193428
nosy: loewis, ned.deily
priority: low
severity: normal
stage: patch review
status: open
title: "xxlimited" extension declared incorrectly in setup.py
versions: Python 3.4

_______________________________________
Python tracker <report at bugs.python.org>
<http://bugs.python.org/issue18517>
_______________________________________