-DG_DISABLE_DEPRECATED and g_io_channel_*()

-DG_DISABLE_DEPRECATED and g_io_channel_*()

[Brian Gix]

Hi All,

There appears to be a number of deprecated g_io_channel_* functions
being called from the bluez tip, which are breaking the build due to
Marcel's latest commit today:

commit effe770cd43cf53fa5842291941dbfc82beab645
Author: Marcel Holtmann <marcel@holtmann.org>
Date:   Tue Jan 18 14:56:31 2011 +0100

    Use -DG_DISABLE_DEPRECATED compiler define


Should this commit be backed out until these deprecated usages have been
fixed, or is this the general way that the issue is forced?

Obviously deprecated functions should not be used, but it creates an
unbuildable bluez in the meantime.

-- 
Brian Gix
bgix@codeaurora.org
Employee of Qualcomm Innovation Center, Inc.
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum

Re: -DG_DISABLE_DEPRECATED and g_io_channel_*()

[Marcel Holtmann]

Hi Brian,

> There appears to be a number of deprecated g_io_channel_* functions
> being called from the bluez tip, which are breaking the build due to
> Marcel's latest commit today:
> 
> commit effe770cd43cf53fa5842291941dbfc82beab645
> Author: Marcel Holtmann <marcel@holtmann.org>
> Date:   Tue Jan 18 14:56:31 2011 +0100
> 
>     Use -DG_DISABLE_DEPRECATED compiler define
> 
> 
> Should this commit be backed out until these deprecated usages have been
> fixed, or is this the general way that the issue is forced?
> 
> Obviously deprecated functions should not be used, but it creates an
> unbuildable bluez in the meantime.

I broke it on purpose to draw some attention to this ;)

Regards

Marcel

Re: -DG_DISABLE_DEPRECATED and g_io_channel_*()

[Luiz Augusto von Dentz]

Hi,

On Wed, Jan 19, 2011 at 1:25 PM, Marcel Holtmann <marcel@holtmann.org> wrot=
e:
> Hi Brian,
>
>> There appears to be a number of deprecated g_io_channel_* functions
>> being called from the bluez tip, which are breaking the build due to
>> Marcel's latest commit today:
>>
>> commit effe770cd43cf53fa5842291941dbfc82beab645
>> Author: Marcel Holtmann <marcel@holtmann.org>
>> Date: =A0 Tue Jan 18 14:56:31 2011 +0100
>>
>> =A0 =A0 Use -DG_DISABLE_DEPRECATED compiler define
>>
>>
>> Should this commit be backed out until these deprecated usages have been
>> fixed, or is this the general way that the issue is forced?
>>
>> Obviously deprecated functions should not be used, but it creates an
>> unbuildable bluez in the meantime.
>
> I broke it on purpose to draw some attention to this ;)

I have just send a pull request to fix those problems.


--=20
Luiz Augusto von Dentz
Computer Engineer