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 --