Issue2148

classification
Title: Remove incomplete ctypes support
Type: Severity: normal
Components: Core Versions: Jython 2.7
process
Status: open Resolution:
Dependencies: Superseder:
Assigned To: zyasoft Nosy List: Arfrever, santa4nt, zyasoft
Priority: normal Keywords:

Created on 2014-05-17.20:14:24 by zyasoft, last changed 2014-05-22.01:01:13 by zyasoft.

Messages
msg8426 (view) Author: Jim Baker (zyasoft) Date: 2014-05-17.20:14:24
ctypes was never completed, and a number of bugs have been raised (#1916, #1917). Incomplete is essentially useless for something like ctypes.

I'm proposing we remove ctypes for beta 4 in favor of eventual CFFI support, specifically through the Jiffy project (current vaporware, https://github.com/jythontools/jiffy). The feedback from our PyPy colleagues is that CFFI is the way to go, especially since it has good unit tests and it's a much smaller project.
msg8437 (view) Author: Arfrever Frehtes Taifersar Arahesis (Arfrever) Date: 2014-05-18.09:26:50
Please do not delete ctypes. Some projects might use these parts of ctypes, which are implemented.
msg8439 (view) Author: Santoso Wijaya (santa4nt) Date: 2014-05-18.17:40:15
Why not just print a warning or something when the incomplete module is imported?
msg8446 (view) Author: Jim Baker (zyasoft) Date: 2014-05-20.17:41:33
Arfrever, are you aware of any such packages? It's hard to see how the current ctypes implementation could actually be useful, vs just a trap for the unwary. But if it's all in use, we can keep and remove as part of Jython 3.x, again in favor of CFFI.

Santoso, assuming we keep it, we could emit a DeprecationWarning describing this package as incomplete.
msg8529 (view) Author: Jim Baker (zyasoft) Date: 2014-05-22.01:01:13
This incomplete support of ctypes is an issue for supporting ipython on Jython. It would be ideal if this did not require a workaround on Jython:
https://github.com/ipython/ipython/pull/4226
History
Date User Action Args
2014-05-22 01:01:13zyasoftsetmessages: + msg8529
2014-05-20 17:41:33zyasoftsetmessages: + msg8446
2014-05-18 17:40:15santa4ntsetnosy: + santa4nt
messages: + msg8439
components: + Core
2014-05-18 09:26:50Arfreversetnosy: + Arfrever
messages: + msg8437
2014-05-17 20:17:55zyasoftsetpriority: normal
assignee: zyasoft
versions: + Jython 2.7
2014-05-17 20:14:24zyasoftcreate