On Saturday 18 of August 2012 16:21:48 Francois Andriot wrote:
Le 18/08/2012 16:16, Slávek Banko a écrit :
François, all,
I noticed that François has in his patches for Fedora incorporated commit
513ffc6e: [tdelibs] Add ability to set KLed off color. Into v3.5.13-sru
branch I am not putting this patch because it breaks the ABI.
I'm interested in your opinion: Add 513ffc6e to v3.5.13-sru or not?
Slavek
Hello,
in 3.5.13-sru, kled is already crashes 3.5.13 non-rebuilt apps because
of a "tqled" => "led" renaming or something like that.
Ex: kpowersave 3.5.13 with tdelibs 3.5.13.1 crashes because of kled.
So I think it won't break it more to introduce this patch too.
Francois
Let me disagree.
Can I mention again that, included "renaming" patches are:
+ Rename old tq methods that no longer need a unique name
+ Remove additional unneeded tq method conversions
+ Rename obsolete tq methods to standard names
+ Rename a few stragglers
+ Fix inadvertent "TQ" changes
These patches change such names, which anyway are in tqtinterface handled
using "#define" - for example: #define tqsetSizePolicy setSizePolicy. These
patches can not break the ABI. And not require recompilation of others.
Exceptions that require recompiling a few other packages I mentioned earlier:
http://trinity-devel.pearsoncomputing.net/?0::9413
513ffc6e patch is different from the above. This patch changes the ABI and
affects a large number of packages, which would have to be recompiled.
Therefore, this patch (though is very small) for me seems with a too big
impact than they should have a patches for SRU.
Slavek
--