Em 05/01/2010, às 02:05, Matt Rogers escreveu:

> On Saturday 02 January 2010 07:31:45 pm Pau Garcia i Quiles wrote:
>> On Sat, Jan 2, 2010 at 9:02 PM, Roman Jarosz <kedge...@gmail.com> wrote:
>>> On Sat, 02 Jan 2010 17:05:28 +0100, Pau Garcia i Quiles
>>> 
>>> <pgqui...@elpauer.org> wrote:
>>>> On Sat, Jan 2, 2010 at 3:28 PM, Roman Jarosz <kedge...@gmail.com> wrote:
>>>>> Hi,
>>>>> 
>>>>> Boiko, here is the first pile of libmsn fixes.
>>>>> 
>>>>> The 4th patch was not included intentionally.
>>>> 
>>>> Regarding patch
>>>> 0005-Remove-ifdef-LIBMSN_INBOX_URL_ENABLED-and-make-pure-.patch, I
>>>> think you do not need to bump soname because the method is virtual.
>>>> See http://tinyurl.com/ydfphzy
>>> 
>>> Hm, I don't see anything about pure virtual functions
>> 
>> From http://tinyurl.com/ydfphzy :
>> 
>> "You cannot...
>> [...]
>> For virtual member functions:
>>    * add a virtual function to a class that doesn't have any virtual
>> functions or virtual bases.
>>    * add new virtual functions to non-leaf classes as this will break
>> subclasses. See below for some workarounds or ask on mailing lists.
>> [...]"
>> 
>> In this case, that's a leaf class and it already has virtual methods
>> (i. e. there is a virtual table already), so the change it's
>> binary-compatible.
>> 
>>> , but the bump is
>>> 
>>> also because
>>> the older version (0.2.0) could have LIBMSN_INBOX_URL_ENABLED on/off so
>>> this
>>> will force Kopete recompilation and will fix all the unexpected crashes.
>> 
>> Soversion was changed from 0.1.0 to 0.2.0 because
>> LIBMSN_INBOX_URL_ENABLED broke binary compatibility even when
>> LIBMSN_INBOX_URL_ENABLED was OFF, due to the addition of data members
>> (search the archives of this mailing list, I explained this in great
>> detail a few weeks ago).
>> 
>> Bump soname just to force Kopete to be rebuilt is a very bad practice
>> and I oppose.
>> 
>> Bumping soname without a need makes my life as the Debian maintainer
>> of libmsn more complicated because I need to submit the package and go
>> through the FTP-masters queue again, which may take several weeks.
>> Please don't.
> 
> Sorry, but we're not responsible for Debian project process issues. We *are* 
> responsible for making sure we don't have stupid bugs and crashes in our 
> code. 
> If it takes a long time to get your stuff uploaded into Debian, that's not 
> our 
> problem.
> 
> I'm in favor of an soversion bump if it makes our job as library users 
> easier, 
> then I'm all for it.

Totally agree. I'm going to apply the soname bump (I just hope Salem won't kill 
me when he is back ;)

Cheers

Boiko
_______________________________________________
kopete-devel mailing list
kopete-devel@kde.org
https://mail.kde.org/mailman/listinfo/kopete-devel

Reply via email to