Bug 1077808

Summary: Error compiling Cython lockdown.pxi file
Product: [Fedora] Fedora Reporter: Menanteau Guy <menantea>
Component: libimobiledeviceAssignee: Peter Robinson <pbrobinson>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: bnocera, cfergeau, pbrobinson
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-05-05 18:56:19 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1051573    

Description Menanteau Guy 2014-03-18 15:37:35 UTC
escription of problem:
Attempting to build the SRPM of libimobiledevice fails with

Making all in cython
make[2]: Entering directory `/home/root/rpmbuild/BUILD/libimobiledevice-1.1.5/cython'
/bin/cython -I/usr/include/plist/cython -I../src -o imobiledevice.c imobiledevice.pyx

Error compiling Cython file:
------------------------------------------------------------
...
        cdef:
            lockdownd_error_t err
            char* c_session_id = NULL
            bint ssl_enabled
            bytes session_id
        err = lockdownd_start_session(self._c_client, host_id, &c_session_id, &ssl_enabled)
                                                                             ^
------------------------------------------------------------

lockdown.pxi:224:78: Cannot assign type 'bool *' to 'int *'
make[2]: *** [imobiledevice.c] Error 1

Version-Release number of selected component (if applicable):
libimobiledevice-1.1.5-3.fc21.ppc64le

How reproducible:
Always, with the above packages. 

Additional info:
A patch already exist in upstream for this bug for libimobiledevice: 

From 62ab50f64fa210304338a5bfccdc6a26fac5f0ec Mon Sep 17 00:00:00 2001
From: "Dawn K. Isabel" <dawn.isabel>
Date: Thu, 30 May 2013 23:36:52 -0400
Subject: [PATCH] cython: Explicitly cast ssl_enabled when calling
 lockdownd_start_session


diff --git a/cython/lockdown.pxi b/cython/lockdown.pxi
index 6fb49a3..ee674b4 100644
--- a/cython/lockdown.pxi
+++ b/cython/lockdown.pxi
@@ -221,7 +221,7 @@ cdef class LockdownClient(PropertyListService):
             char* c_session_id = NULL
             bint ssl_enabled
             bytes session_id
-        err = lockdownd_start_session(self._c_client, host_id, &c_session_id, &ssl_enabled)
+        err = lockdownd_start_session(self._c_client, host_id, &c_session_id, <int *>&ssl_enabled)
         try:
             self.handle_error(err)

Comment 1 Menanteau Guy 2014-03-18 15:57:31 UTC
Cython version was 0.19.2-2

Comment 2 Peter Robinson 2014-04-13 11:57:34 UTC
Should be fixed with new release that I should land shortly