By default, Bugzilla does not search the list of RESOLVED bugs.
You can force it to do so by putting the upper-case word ALL in front of your search query, e.g.: ALL tdelibs
We recommend searching for bugs this way, as you may discover that your bug has already been resolved and fixed in a later release.

Bug 1922

Summary:	tdeutils superkaramba FTBFS with python 3
Product:	TDE	Reporter:	David C. Rankin <trin>
Component:	tdeutils	Assignee:	Timothy Pearson <kb9vqf>
Status:	RESOLVED FIXED
Severity:	normal	CC:	bugwatch, kb9vqf, trin
Priority:	P5
Version:	R14.0.0 [Trinity]
Hardware:	All
OS:	Linux
Compiler Version:		TDE Version String:
Application Version:		Application Name:
Bug Depends on:
Bug Blocks:	2014
Attachments:	patch for tdeutils allowing superkaramba to build on python 3.x

Description David C. Rankin 2014-02-09 21:28:41 CST

Created attachment 1924 [details]
patch for tdeutils allowing superkaramba to build on python 3.x

tdeutils superkaramba fails to build from source if built with python >=3. The patch provided allows superkaramba to build on both python 2.x and python 3.x. The changes that needed to support both versions are:

PyString_CheckExact         ->  PyBytes_CheckExact
PyString_FromString         ->  PyBytes_FromString
PyString_FromStringAndSize  ->  PyBytes_FromStringAndSize
PyInt_FromLong              ->  PyLong_FromLong

The difficult part of the port requiring preprocessor checks surrounds initializing the karamba python module. python 2.x Py_InitModule has been replace with a new initialization scheme in python 3.x involving PyModuleDef and PyModule_Create. The following patch was created with reference to:

http://docs.python.org/3/howto/cporting.html

The patch will need to be verified before being pushed. It builds fine on bleeding-edge arch.

Comment 1 Timothy Pearson 2014-09-27 11:43:43 CDT

Comment on attachment 1924 [details]
patch for tdeutils allowing superkaramba to build on python 3.x

Patch looks sane; superkaramba builds and functions with Python 3.4.

Pushed to GIT in hash 36230c.

Thanks for reporting, and for the patch!