bluez-tools FTBFS on wily-proposed in 64-bit archs due to implicit pointer conversion

Bug #1490204 reported by Tiago Stürmer Daitx
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
bluez-tools (Debian)
Fix Released
Unknown
bluez-tools (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

After LDADD fix (LP: #1489661, Debian #797128) bluez-tools build now fails on 64-bit systems due to implicit pointer conversion in bt-device.c.

Build logs at: https://launchpad.net/ubuntu/+source/bluez-tools/0.2.0~20140808-3ubuntu1

gcc -DHAVE_CONFIG_H -I. -I.. -I/usr/include/glib-2.0 -I/usr/lib/powerpc64le-linux-gnu/glib-2.0/include -pthread -I/usr/include/glib-2.0 -I/usr/lib/powerpc64le-linux-gnu/glib-2.0/include -D_FORTIFY_SOURCE=2 -g -O3 -fstack-protector-strong -Wformat -Werror=format-security -c -o bt-device.o bt-device.c
bt-device.c: In function '_bt_device_sdp_browse':
bt-device.c:345:50: warning: implicit declaration of function 'g_unix_input_stream_new' [-Wimplicit-function-declaration]
     GInputStream *exec_output = (GInputStream *) g_unix_input_stream_new(pipefd[0], TRUE);

Revision history for this message
Tiago Stürmer Daitx (tdaitx) wrote :

This patch provides the right includes so gcc can know the right return type instead of assuming an integer. This fix prevents the implicit pointer conversion warning from happening.

bt-device.c was missing an include for gio/gunixinputstream.c and configure.ac had to be updated to include gio-unix-2.0 module.

tags: added: ftbfs patch patch-forwarded-debian patch-forwarded-upstream
Revision history for this message
Tiago Stürmer Daitx (tdaitx) wrote :
Changed in bluez-tools (Debian):
status: Unknown → New
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package bluez-tools - 0.2.0~20140808-3ubuntu2

---------------
bluez-tools (0.2.0~20140808-3ubuntu2) wily; urgency=medium

  * debian/patches/fix-implicity-pointer-conversion.patch: include
    gio-unix-2.0 module to configure.ac and gio/gunixinputstream.h
    to bt-device.c (LP: #1490204, Closes: 797356)

 -- Tiago Stürmer Daitx <email address hidden> Sat, 29 Aug 2015 18:30:37 -0300

Changed in bluez-tools (Ubuntu):
status: New → Fix Released
Changed in bluez-tools (Debian):
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.