Importing ctypes.wintypes on Linux gives a ValueError instead of an ImportError

[techtonik]

BPO	16396
Nosy	@amauryfa, @jaraco, @abalkin, @tiran, @benjaminp, @ezio-melotti, @meadori, @serhiy-storchaka, @zooba, @miss-islington, @hauntsaninja
PRs	bpo-16396: Allow wintypes to be imported on non-Windows systems. #21394 [3.9] bpo-16396: Allow wintypes to be imported on non-Windows systems. (GH-21394) #22790 [3.8] bpo-16396: Allow wintypes to be imported on non-Windows systems. (GH-21394) #22791 bpo-16396: fix BPO number in changelog #23951 [3.9] bpo-16396: fix BPO number in changelog (GH-23951) #23956
Files	16396_vbool.patch

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = 'https://github.com/benjaminp'
closed_at = <Date 2020-10-19.22:07:07.341>
created_at = <Date 2012-11-03.15:22:51.972>
labels = ['ctypes', 'type-bug', '3.8', '3.9', '3.10']
title = 'Importing ctypes.wintypes on Linux gives a ValueError instead of an ImportError'
updated_at = <Date 2020-12-29.11:52:15.433>
user = 'https://bugs.python.org/techtonik'

bugs.python.org fields:

activity = <Date 2020-12-29.11:52:15.433>
actor = 'serhiy.storchaka'
assignee = 'benjamin.peterson'
closed = True
closed_date = <Date 2020-10-19.22:07:07.341>
closer = 'steve.dower'
components = ['ctypes']
creation = <Date 2012-11-03.15:22:51.972>
creator = 'techtonik'
dependencies = []
files = ['30684']
hgrepos = []
issue_num = 16396
keywords = ['patch']
message_count = 22.0
messages = ['174634', '174635', '188245', '188707', '191756', '191763', '192324', '194927', '194948', '194985', '195652', '195675', '195825', '195924', '195926', '195940', '284191', '379033', '379042', '379047', '383808', '383983']
nosy_count = 14.0
nosy_names = ['amaury.forgeotdarc', 'jaraco', 'belopolsky', 'techtonik', 'christian.heimes', 'benjamin.peterson', 'ezio.melotti', 'meador.inge', 'serhiy.storchaka', 'steve.dower', 'dmi.baranov', 'Mike Place', 'miss-islington', 'hauntsaninja']
pr_nums = ['21394', '22790', '22791', '23951', '23956']
priority = 'normal'
resolution = 'fixed'
stage = 'resolved'
status = 'closed'
superseder = None
type = 'behavior'
url = 'https://bugs.python.org/issue16396'
versions = ['Python 3.8', 'Python 3.9', 'Python 3.10']

[techtonik]

>>> import ctypes.wintypes
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/lib/python3.2/ctypes/wintypes.py", line 20, in <module>
    class VARIANT_BOOL(ctypes._SimpleCData):
ValueError: _type_ 'v' not supported
>>> 

Shouldn't it just import silently without failing? Or if it's destined to fail, explain how to make a cross-platform import?

[techtonik]

Perhaps the patch already there - see http://www.themacaque.com/?p=826

[dmibaranov]

Found only this "patch" [1] :) I think is possible to change VARIANT_BOOL._type_ to any of short types [2] for non-"nt" platforms?

[1] https://code.launchpad.net/~mandel/python-distutils-extra/import_issues/+merge/53519
[2] http://msdn.microsoft.com/en-us/library/cc237864.aspx

[ezio-melotti]

That patch is more a workaround than an actual fix. Lib/ctypes/wintypes.py should either fail with an ImportError or be importable. For the former it's possible to catch the ValueError and turn it into an ImportError, or perhaps raise it if some precondition is missing; for the latter, either the creation of that signle class is skipped if _type_ 'v' is not supported, or a way to define it that works on other platforms too should be found instead.

[tiran]

The fix is trivial:

• define VARIANT_FALSE and VARIANT_BOOL according to specs
• enable 'v' on non-Windows systems
• enable tests for vbool

Done

[jaraco]

Looks good to me. Ben, any objections to applying this to 2.7?

[tiran]

RM, please decide. :)

[benjaminp]

Smells like a new feature to me.

[ezio-melotti]

Even if the patch is applied only on 3.4, I would still like to see the ValueError turned into ImportError for 2.7/3.3.

[jaraco]

My sense on the issue is that wintypes was added to the library and was never intended to raise a ValueError on import. By that logic, the behavior is a bug, not a new feature. I agree with Ezio that raising a ValueError on import is a bug. And since the patch not only addresses the ValueError on import, but simply addresses the underlying cause, it seems to me the most obvious solution.

[amauryfa]

I think it's the opposite: when Unix support was added to ctypes, 'import ctypes.wintypes' was not considered. By that logic, the patch is a new feature.
IMO "historical" arguments are moot :-)

I agree with the conclusion tough: the patch will not break code that carefully catches ValueError, and so it suitable for 2.7.

[meadori]

On Mon, Aug 12, 2013 at 7:11 AM, Ezio Melotti <report@bugs.python.org>wrote:

Even if the patch is applied only on 3.4, I would still like to see the
ValueError turned into ImportError for 2.7/3.3.

Why not raise an ImportError for all versions? I don't see how
'ctypes.wintypes' is ever actually useful on anything but Windows.
Allowing it to successfully import on non-Windows systems seems misleading.