[Bug 458975] Review Request: Pivy - Coin binding for Python

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458975





--- Comment #11 from Ralf Corsepius <rc040203 at freenet.de>  2009-07-29 12:03:41 EDT ---
FWIW: It's pretty easy to work around the getVersionToolkitString issue in
Pivy:

Simply somehow hard-code PYQT_MODULE_IMPORT_NAME in Pivy/interfaces/soqt.i,
instead of applying dynamic detection (As we only have one SoQt in Fedora, we
don't need it anyway).

One way to achieve this:

diff -Naur Pivy.orig/interfaces/soqt.i Pivy/interfaces/soqt.i
--- Pivy.orig/interfaces/soqt.i 2007-12-14 01:35:37.000000000 +0100
+++ Pivy/interfaces/soqt.i      2009-07-29 17:37:37.000000000 +0200
@@ -74,12 +74,16 @@
   /* determine the Qt version SoQt has been compiled with for correct
    * PyQt module import */
   if (!PYQT_MODULE_IMPORT_NAME) {
+#if 0     
     PYQT_MODULE_IMPORT_NAME = "qt";

     if (strlen(SoQt::getVersionToolkitString()) >= 1 &&
        SoQt::getVersionToolkitString()[0] == '4') {
+#endif
       PYQT_MODULE_IMPORT_NAME = "PyQt4.Qt";
+#if 0
     }
+#endif
   }
 }
 %}

Alternatively, one could initialize
PYQT_MODULE_IMPORT_NAME to "PyQt4.Qt"
at the very beginning of Pivy/interfaces/soqt.i

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.